Add a PhysicalWebIgnoreOtherClients feature

Add a PhysicalWebIgnoreOtherClients feature

The Physical Web feature tests to see if other notification-based
Physical Web clients exist on the device before surfacing
notifications.  This feature, when enabled, will bypass that check.

BUG=616316
Committed: https://crrev.com/4159c8005f3f28b5d4381e6d37b8c33c3b1542ef
Cr-Commit-Position: refs/heads/master@{#402329}

[cco3, 2016-06-16 00:49:38]

cco3@chromium.org changed reviewers:
+ mattreynolds@chromium.org

[cco3, 2016-06-23 19:05:24]

ping

[mattreynolds, 2016-06-24 17:52:23]

lgtm

[cco3, 2016-06-24 17:59:01]

cco3@chromium.org changed reviewers:
+ nyquist@chromium.org

[cco3, 2016-06-24 17:59:16]

cco3@chromium.org changed reviewers:
+ rkaplow@chromium.org

[nyquist, 2016-06-24 19:13:52]

lgtm

[cco3, 2016-06-27 16:35:47]

The CQ bit was checked by cco3@chromium.org

[commit-bot: I haz the power, 2016-06-27 16:36:14]

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, 2016-06-27 16:38:59]

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

[commit-bot: I haz the power, 2016-06-27 16:39:00]

Try jobs failed on following builders:
  ios-simulator-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-gn/...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[cco3, 2016-06-27 17:19:26]

The CQ bit was checked by cco3@chromium.org

[cco3, 2016-06-27 17:19:26]

The patchset sent to the CQ was uploaded after l-g-t-m from
nyquist@chromium.org, mattreynolds@chromium.org
Link to the patchset: https://codereview.chromium.org/2071713002/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2016-06-27 17:20:08]

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, 2016-06-27 18:16:31]

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

[commit-bot: I haz the power, 2016-06-27 18:16:32]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[rkaplow, 2016-06-27 18:45:42]

lgtm

[cco3, 2016-06-27 19:03:41]

Matt, could you take a second look at the java files?

[cco3, 2016-06-27 19:03:41]

Matt, could you take a second look at the java files?

[cco3, 2016-06-27 21:47:14]

The CQ bit was checked by cco3@chromium.org

[cco3, 2016-06-27 21:47:15]

The patchset sent to the CQ was uploaded after l-g-t-m from
nyquist@chromium.org, rkaplow@chromium.org, mattreynolds@chromium.org
Link to the patchset: https://codereview.chromium.org/2071713002/#ps60001
(title: "Store setting in onChromeStart")

[commit-bot: I haz the power, 2016-06-27 21:48:10]

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

[mattreynolds, 2016-06-27 21:48:59]

https://codereview.chromium.org/2071713002/diff/40001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWeb.java
(right):

https://codereview.chromium.org/2071713002/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWeb.java:66:
boolean ignoreOtherClients =
ChromeFeatureList.isEnabled(IGNORE_OTHER_CLIENTS_FEATURE_NAME);
Let's move this to onChromeStart since the feature can't be toggled without
restarting Chrome.

[commit-bot: I haz the power, 2016-06-27 23:26:26]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-06-27 23:29:25]

Description was changed from

==========
Add a PhysicalWebIgnoreOtherClients feature

The Physical Web feature tests to see if other notification-based
Physical Web clients exist on the device before surfacing
notifications.  This feature, when enabled, will bypass that check.

BUG=616316
==========

to

==========
Add a PhysicalWebIgnoreOtherClients feature

The Physical Web feature tests to see if other notification-based
Physical Web clients exist on the device before surfacing
notifications.  This feature, when enabled, will bypass that check.

BUG=616316

Committed: https://crrev.com/4159c8005f3f28b5d4381e6d37b8c33c3b1542ef
Cr-Commit-Position: refs/heads/master@{#402329}
==========

[commit-bot: I haz the power, 2016-06-27 23:29:26]

Patchset 4 (id:??) landed as
https://crrev.com/4159c8005f3f28b5d4381e6d37b8c33c3b1542ef
Cr-Commit-Position: refs/heads/master@{#402329}