Fix feature flag race condition

Previously, a test that should have failed passed when uri=null, because
none of the test groups had uri patterns to check against. Make the
test groups more realistic by adding in the scopes attribute for at
least one of them.

In cases like: https://hypothes.is/a/rGSnNwCKSuiG3VT__k3VvA where the
sidebar is not being used, the document uri is set to null in the groups
service and passed to the combineGroups function. This means that the
isScopedToUri logic must handle the case where uri=null.

Since the feature flag isn't available until the profile api request
completes, and the profile api request is run asynchronously with the
group api requests, it doesn't make sense to load conditionaly. Instead,
call all the group api requests and filter the groups for the old groups
menu later. In this case, filtering is done in the groups.all
conditionally based on the feature flag.

Since the isScopedToUri is no longer dependent on scope enforcement
and scope enforcement is now taken into account as part of whether
a group is selectable or not, disableOosGroupSelection is no longer
necessary. Group selection can be determined purely from isScopedToUri
and scopes.enforced.

Previously, if the scope was not enforced, isScopedToUri would always
return true. Now, isScopedToUri is true only when the scopes.uri_patterns
match.

getInScopeGroups returns the list of groups for the old groups
menu. Aka: only groups that are in scope are returned.

Revert "Remove feature flag from group service"

Remove feature flag from group service

Convert `<svg-icon>` component to React

Merge pull request #1055 from hypothesis/dependabot/npm_and_yarn/babel-plugin-transform-async-to-promises-0.8.9

Bump babel-plugin-transform-async-to-promises from 0.8.8 to 0.8.9

Bump mocha from 6.1.1 to 6.1.2

Bump enzyme-adapter-preact-pure from 1.10.3 to 1.10.4

Bumps [babel-plugin-transform-async-to-promises](https://github.com/rpetrich/babel-plugin-transform-async-to-promises) from 0.8.8 to 0.8.9.
- [Release notes](https://github.com/rpetrich/babel-plugin-transform-async-to-promises/releases)
- [Commits](https://github.com/rpetrich/babel-plugin-transform-async-to-promises/compare/v0.8.8...v0.8.9)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Bumps [mocha](https://github.com/mochajs/mocha) from 6.1.1 to 6.1.2.
- [Release notes](https://github.com/mochajs/mocha/releases)
- [Changelog](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md)
- [Commits](https://github.com/mochajs/mocha/compare/v6.1.1...v6.1.2)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Bumps [enzyme-adapter-preact-pure](https://github.com/robertknight/enzyme-adapter-preact-pure) from 1.10.3 to 1.10.4.
- [Release notes](https://github.com/robertknight/enzyme-adapter-preact-pure/releases)
- [Changelog](https://github.com/robertknight/enzyme-adapter-preact-pure/blob/master/CHANGELOG.md)
- [Commits](https://github.com/robertknight/enzyme-adapter-preact-pure/commits)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Previously, isScopedToUri took into account enforcement but now
it does not. Thus, isSelectable logic based on scopes.enforced
and isScopedToUri was added to the group-list-section controller
to determine whether a group should be out of scope/not selectable.

Since the feature flag isn't available until the profile api request
completes, and the profile api request is run asynchronously with the
group api requests, it doesn't make sense to load conditionaly. Instead,
call all the group api requests and filter the groups for the old groups
menu later. In this case, filtering is done in the groups.all
conditionally based on the feature flag.

Previously, if the scope was not enforced, isScopedToUri would always
return true. Now, isScopedToUri is true only when the scopes.uri_patterns
match.

getInScopeGroups returns the list of groups for the old groups
menu. Aka: only groups that are in scope are returned.

Convert `<annotation-action-button>` component to React

Bump @babel/preset-env from 7.4.2 to 7.4.3

Bumps [@babel/preset-env](https://github.com/babel/babel) from 7.4.2 to 7.4.3.
- [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.2...v7.4.3)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Bump @octokit/rest from 16.22.0 to 16.23.2

Bump @babel/core from 7.4.0 to 7.4.3

Bump enzyme-adapter-preact-pure from 1.10.2 to 1.10.3

Bump preact from 10.0.0-alpha.2 to 10.0.0-alpha.4

Bump codecov from 3.2.0 to 3.3.0

Bump mocha from 6.0.2 to 6.1.1

Bump commander from 2.19.0 to 2.20.0

Bump autoprefixer from 9.5.0 to 9.5.1

Bumps [commander](https://github.com/tj/commander.js) from 2.19.0 to 2.20.0.
- [Release notes](https://github.com/tj/commander.js/releases)
- [Changelog](https://github.com/tj/commander.js/blob/master/CHANGELOG.md)
- [Commits](https://github.com/tj/commander.js/compare/v2.19.0...v2.20.0)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Bumps [mocha](https://github.com/mochajs/mocha) from 6.0.2 to 6.1.1.
- [Release notes](https://github.com/mochajs/mocha/releases)
- [Changelog](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md)
- [Commits](https://github.com/mochajs/mocha/compare/v6.0.2...v6.1.1)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 16.22.0 to 16.23.2.
- [Release notes](https://github.com/octokit/rest.js/releases)
- [Commits](https://github.com/octokit/rest.js/compare/v16.22.0...v16.23.2)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Bumps [codecov](https://github.com/codecov/codecov-node) from 3.2.0 to 3.3.0.
- [Release notes](https://github.com/codecov/codecov-node/releases)
- [Commits](https://github.com/codecov/codecov-node/commits/v3.3.0)
Signed-off-by: dependabot[bot] <support@dependabot.com>