Fix annotation test in light of search merge

The visual search merge made it so that annotations have two body
elements, one which shows the normal body and the other which shows
the body with highlighted search terms.

The annotation test was selecting the wrong one (and complaining that
it was not visible).

@gergely-ujvari: you owe me a :beer:
Correct the position of the controlbar "top"
Escape a '>' with &gt;
Autofocus search input on click
whitespace
Avoid flicker on the control bar tab
For human parsing, positive > negative conditonals
Add make clean target
Run tests maximized
Skip an unneccessary navigation in tests
Avoid sleeps in tests by leveraging implicit_wait

If we use the class name of the sidebar frame for the context manager
then we can let the driver's implicit_wait take care of delaying until
the app has loaded. This makes the tests run much faster.
Refactor functional tests

This introduces some changes to make the functional tests easier and
faster.

- Have the context manager ensure the sidebar is open.
- Use class names wherever possible to leverage implicit_wait.
- Use refresh rather than navigating to Google in test_annotation.
- Improve login method form use and add similar register method. This
  changed involved renaming `login` to `register` first since that is
  what it was actually doing anyway.