[Reland][Extensions Bindings] Add support for filtered events

[Reland][Extensions Bindings] Add support for filtered events

Some extension API events support "filtering", which allows extensions
to only receive event notifications for a subset of the total events.
For instance, the webNavigation API allows an extension to specify
a set of url filters in order to specify which urls the listener is
interested in, and not be notified of every navigation.

Expand the native event bindings to support filtering for applicable
events. Abstract out an APIEventListeners class to handle adding,
removing, and querying event listeners, including retrieving those
for certain filtering info. This currently has two implementations -
FilteredEventListeners and UnfilteredFilteredEventListeners.
Add tests for the same.

Expand the end-to-end test with webNavigation events to test filtered
events.

Fix for reland: change the url in the web navigation test and allow old
urls from previous test cases.

BUG=653596
Review-Url: https://codereview.chromium.org/2768093002
Cr-Commit-Position: refs/heads/master@{#458962}
Committed: https://chromium.googlesource.com/chromium/src/+/ddc50bcb9cd192d117d23e025ce39c5a4da33e19

[Devlin, 2017-03-22 20:23:27]

The CQ bit was checked by rdevlin.cronin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-22 20:24:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-22 20:55:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-22 20:55:39]

Dry run: Try jobs failed on following builders:
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[Devlin, 2017-03-22 21:10:18]

rdevlin.cronin@chromium.org changed reviewers:
+ lazyboy@chromium.org

[Devlin, 2017-03-22 21:10:19]

lazyboy@, mind taking a look?  It seems like the tests were failing because
other tabs could still be updating (and thus throwing off our expectations).
I've relaxed them a little.

The original patch is PS 1, PS 2 has the fix.

jbroman@, FYI.

[lazyboy, 2017-03-22 21:13:46]

chrome/test/data/extensions/api_test/native_bindings/extension/background.js
changes lgtm

[Devlin, 2017-03-22 22:40:55]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2017-03-22 22:41:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-22 23:45:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-22 23:45:57]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Devlin, 2017-03-22 23:47:00]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2017-03-22 23:48:00]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-23 00:56:01]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490226420627080,
"parent_rev": "69ae14a2e56b0ee3642bd5961fe76704848f68e3", "commit_rev":
"ddc50bcb9cd192d117d23e025ce39c5a4da33e19"}

[commit-bot: I haz the power, 2017-03-23 00:57:23]

Description was changed from

==========
[Reland][Extensions Bindings] Add support for filtered events

Some extension API events support "filtering", which allows extensions
to only receive event notifications for a subset of the total events.
For instance, the webNavigation API allows an extension to specify
a set of url filters in order to specify which urls the listener is
interested in, and not be notified of every navigation.

Expand the native event bindings to support filtering for applicable
events. Abstract out an APIEventListeners class to handle adding,
removing, and querying event listeners, including retrieving those
for certain filtering info. This currently has two implementations -
FilteredEventListeners and UnfilteredFilteredEventListeners.
Add tests for the same.

Expand the end-to-end test with webNavigation events to test filtered
events.

Fix for reland: change the url in the web navigation test and allow old
urls from previous test cases.

BUG=653596
==========

to

==========
[Reland][Extensions Bindings] Add support for filtered events

Some extension API events support "filtering", which allows extensions
to only receive event notifications for a subset of the total events.
For instance, the webNavigation API allows an extension to specify
a set of url filters in order to specify which urls the listener is
interested in, and not be notified of every navigation.

Expand the native event bindings to support filtering for applicable
events. Abstract out an APIEventListeners class to handle adding,
removing, and querying event listeners, including retrieving those
for certain filtering info. This currently has two implementations -
FilteredEventListeners and UnfilteredFilteredEventListeners.
Add tests for the same.

Expand the end-to-end test with webNavigation events to test filtered
events.

Fix for reland: change the url in the web navigation test and allow old
urls from previous test cases.

BUG=653596

Review-Url: https://codereview.chromium.org/2768093002
Cr-Commit-Position: refs/heads/master@{#458962}
Committed:
https://chromium.googlesource.com/chromium/src/+/ddc50bcb9cd192d117d23e025ce3...
==========

[commit-bot: I haz the power, 2017-03-23 00:57:24]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/ddc50bcb9cd192d117d23e025ce3...