This reverts commit 20dca37ee333684eb8f86bee722e4d308acea424.

Relates to #1200

1200 auth fixes

Relates to #1200

Relates to #1200

Respond to view value changes by watching the view value on the scope
rather than registering a view change listener. View change listeners
only fire on changes which are valid.

Be careful not to use the directive in combination with ng-hide or
ng-show since the CSS changes subvert the transition.

Improve the conditions for showing/hiding so that it considers the
validity of a field in addition to whether the field is pristine or
dirty.

Relates to #1200

Relates to #1200

Partially revert the changes made in https://github.com/hypothesis/h/commit/c840a620fc040ceb7941c9b50a379b9c271b0728. Fixes #1201.

When annotations are loaded/updated in either side, they will be
automatically transmitted to the other side by the bridbe plugin.

Then on the other side, the new data is merged into any existing one.

The problem is that when doing this merge, some fields (which are objects
themselves) should not be overwritten; they should be merged, too.

This change adds selective handling of the `target` field, which is
a list of objects. Now each individual target is merged, ever overwritten.

Furthermore, there are a few special cases when some fields
(inside targets) should not be updated at all.

Now those updates are refused.

Fixes #1198.

Close #468

Close #1001

Convert {username, provider} objects to strings.

Overhaul horus view handling

Refactor the views module to limit code exposure to horus.

- Implement a view mapper that acts on the horus class-based form
  views to coerce the result to a dictionary. This mapper pulls most
  of the request and response munging away from the horus view logic
  overrides, which are left as simple subclasses. Page views for horus
  now fire login/logout events and work around eventray/horus#43 for
  password resets.
- Improve error reporting by showing flash messages only for colander
  errors, letting horus handle the rest. Remove corresponding hack in
  client-side flash handling.

IE doesn't have the document.baseURI property.
This workaround constructs the baseURI if not present.

This reverts commit most of 636651959f26671098e692a28fde147a835f6f23.
(I have left the changes to h/js/flash.coffee )

Close #479

Includes backport of openannotation/annotator#200 fix. Lines removed
from flash.coffee as they are now obsolete.

Now we have the potential for multiple sign-in.

Revisit the language of the auth tab blocks to remove the redundant
forgot and claim views.

Make the userName filter more useful by expanding it to a general
parameterized persona filter.

Don't send a new search request until the logout request has returned.

Instead of sending a token with the application view (which is really
more of a session state view), fetch it separately using the tokenUrl
option of the Auth plugin. Account for the case where known personas
could change in the future without invalidating the current persona
and reset the Auth plugin whenever the persona in use changes. Use the
persona parameter to the token endpoint to get the appropriate token.

The refresh function is not defined in the scope it's called in which
is why the event is necessary. However, unlike the original, the app
scope is used instead of the root scope because there's no need to
dispatch from the root scope here.

Since the token changes whenever the list of personas changes just
refactor the logic in the token watcher to cause exactly one search
to happen when the token changes without consulting a separate
property, simply by checking the tool state.

This reverts commit ca43ccc1d9559476f95c714c1b055194f03dd0c2.

These should have been added along with the rest in 8c7db59e05dd7757989f6b31a19edef6cfb686fd

Since the highlight code uses jQuery's .offset() we don't need to
factor in the wrapper offset.

Close #931

The root core of problem #1179 is actually an upstream problem. Our code is only necessary until we can consume the fixed version of annotator.