Bump eslint from 5.16.0 to 6.0.1

Bump @babel/preset-env from 7.4.5 to 7.5.0

 - Suppress all warnings about use of `hasOwnProperty`. Some of these
   warnings *are* valid issues. I've decided not to block the ESLint
   upgrade on them however.
 - Suppress two other specific warnings

Merge with `yarn.lock` updates on master.

This is the result of running:

    yarn upgrade -P @babel/core @babel/preset-env

Convert `<top-bar>` component to (P)react

 - Clarify or reword a few comments and test descriptions

 - Pluralize "annotations" correctly in the real-time update notification
   icon's tooltip

Fix rendering bug in moderation-banner component

Bump autoprefixer from 9.6.0 to 9.6.1

Bumps [@babel/preset-env](https://github.com/babel/babel) from 7.4.5 to 7.5.0.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/master/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/compare/v7.4.5...v7.5.0)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [eslint](https://github.com/eslint/eslint) from 5.16.0 to 6.0.1.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v5.16.0...v6.0.1)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bump unorm from 1.5.0 to 1.6.0

Bump @babel/core from 7.4.5 to 7.5.0

Bump browserify from 16.2.3 to 16.3.0

Bump websocket from 1.0.28 to 1.0.29

Bumps [unorm](https://github.com/walling/unorm) from 1.5.0 to 1.6.0.
- [Release notes](https://github.com/walling/unorm/releases)
- [Commits](https://github.com/walling/unorm/compare/v1.5.0...v1.6.0)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [autoprefixer](https://github.com/postcss/autoprefixer) from 9.6.0 to 9.6.1.
- [Release notes](https://github.com/postcss/autoprefixer/releases)
- [Changelog](https://github.com/postcss/autoprefixer/blob/master/CHANGELOG.md)
- [Commits](https://github.com/postcss/autoprefixer/compare/9.6.0...9.6.1)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [browserify](https://github.com/browserify/browserify) from 16.2.3 to 16.3.0.
- [Release notes](https://github.com/browserify/browserify/releases)
- [Changelog](https://github.com/browserify/browserify/blob/master/changelog.markdown)
- [Commits](https://github.com/browserify/browserify/compare/v16.2.3...v16.3.0)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [@babel/core](https://github.com/babel/babel) from 7.4.5 to 7.5.0.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/master/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/compare/v7.4.5...v7.5.0)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [websocket](https://github.com/theturtle32/WebSocket-Node) from 1.0.28 to 1.0.29.
- [Release notes](https://github.com/theturtle32/WebSocket-Node/releases)
- [Changelog](https://github.com/theturtle32/WebSocket-Node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/theturtle32/WebSocket-Node/compare/v1.0.28...v1.0.29)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Convert `AnnotationHeader` and its allies to preact

Previously, this component rendered either a `div` or an `a` depending
on context. This makes styling and whatnot impractical.

Preact exhibits funny behavior for inline conditionals that don’t resolve to an actual boolean value. When the flagCount variable is 0 it renders the conditional statement directly in the dom. Casting to a boolean prevents this.

Move the CSS styles into the module that corresponds to the component
where they are used.

Instead of passing a `searchController` prop into `TopBar` which is
responsible for reading and updating the search query, from different
sources depending on whether this is the sidebar or stream, handle
reading/updating the search from within `TopBar` itself:

 - In the sidebar, read/update the filter query in the store
 - In the stream, render a `StreamSearchInput` control which
   reads/updates the "$q" param from the Angular `$location` service

This removes some indirection and makes it easier to follow where the
search input gets its data from and what happens when it is changed.

Also fix a bug where existing annotations were not cleared after a route
change from `/a/:id` to `/stream?q={query}`.

This provides more runtime-checked documentation about what these props
look like.

If there was not enough space for all of the items in the top bar, the
group name could end up wrapping. This tended to happen when the group
name was long, the sidebar was narrow and the search input was active.

Since this component is no longer referenced within an Angular template,
the arguments no longer have to be named as properties in an object.

 - Replace `h-branding` with `applyTheme` utility

 - Replace `h-tooltip` with standard tooltips using the `title`
   attribute, which matches other icons in the top bar

 - Fix a double-quoted string after `<svg-icon>` in an Angular template
   was converted to `<SvgIcon ...>` in JSX

 - Convert `<top-bar>` component to Preact
 - Add additional tests for top bar in different states
 - Remove `<login-control>` component. This has been inlined into `TopBar`
 - Remove unused component wrappers for components that are no longer
   referenced from Angular templates

Only emit end if searchClient is done searching

- Used by `AnnotationHeader`
- Renders group and privacy info for an annotation

- Used by `AnnotationHeader`
- Renders metadata about an annotation’s document

Add self-closing ESlint enforcement