Modify StartPhysicalWebDiscovery to take an ios::ChromeBrowserState param

Modify StartPhysicalWebDiscovery to take an ios::ChromeBrowserState param

In a future CL, the Physical Web's auto-optin behavior will be modified to
require that Google is configured as the default search provider. This CL
adds a new overload for StartPhysicalWebDiscovery so downstream users can
be modified to use the new signature that provides access to the browser
state, from which we can determine the default search provider.

BUG=630769
Committed: https://crrev.com/7d1e59e370b8810d5ba953d38887ed4a79af4a5e
Cr-Commit-Position: refs/heads/master@{#438003}

[mattreynolds, 2016-12-09 21:53:16]

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

[mattreynolds, 2016-12-09 21:53:16]

Hi Sylvain, PTAL

[sdefresne, 2016-12-11 09:00:00]

lgtm

[mattreynolds, 2016-12-12 18:19:07]

The CQ bit was checked by mattreynolds@chromium.org

[commit-bot: I haz the power, 2016-12-12 18:19:27]

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-12-12 18:23:30]

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

[commit-bot: I haz the power, 2016-12-12 18:23:32]

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

[mattreynolds, 2016-12-12 19:52:39]

The CQ bit was checked by mattreynolds@chromium.org

[mattreynolds, 2016-12-12 19:52:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
sdefresne@chromium.org
Link to the patchset: https://codereview.chromium.org/2566753002/#ps20001
(title: "fix deps")

[commit-bot: I haz the power, 2016-12-12 19:53:26]

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-12-12 23:06:08]

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

[commit-bot: I haz the power, 2016-12-12 23:06:09]

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...)

[mattreynolds, 2016-12-12 23:51:28]

The CQ bit was checked by mattreynolds@chromium.org

[commit-bot: I haz the power, 2016-12-12 23:52:28]

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-12-13 02:53:23]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481586688105550,
"parent_rev": "3a660de3199944f53e79d62055aaab5925ef16de", "commit_rev":
"306cb6c2d51bd7664dc1523f94abb2eff4c71479"}

[commit-bot: I haz the power, 2016-12-13 02:53:48]

Description was changed from

==========
Modify StartPhysicalWebDiscovery to take an ios::ChromeBrowserState param

In a future CL, the Physical Web's auto-optin behavior will be modified to
require that Google is configured as the default search provider. This CL
adds a new overload for StartPhysicalWebDiscovery so downstream users can
be modified to use the new signature that provides access to the browser
state, from which we can determine the default search provider.

BUG=630769
==========

to

==========
Modify StartPhysicalWebDiscovery to take an ios::ChromeBrowserState param

In a future CL, the Physical Web's auto-optin behavior will be modified to
require that Google is configured as the default search provider. This CL
adds a new overload for StartPhysicalWebDiscovery so downstream users can
be modified to use the new signature that provides access to the browser
state, from which we can determine the default search provider.

BUG=630769

Review-Url: https://codereview.chromium.org/2566753002
==========

[commit-bot: I haz the power, 2016-12-13 02:53:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-13 02:56:58]

Description was changed from

==========
Modify StartPhysicalWebDiscovery to take an ios::ChromeBrowserState param

In a future CL, the Physical Web's auto-optin behavior will be modified to
require that Google is configured as the default search provider. This CL
adds a new overload for StartPhysicalWebDiscovery so downstream users can
be modified to use the new signature that provides access to the browser
state, from which we can determine the default search provider.

BUG=630769

Review-Url: https://codereview.chromium.org/2566753002
==========

to

==========
Modify StartPhysicalWebDiscovery to take an ios::ChromeBrowserState param

In a future CL, the Physical Web's auto-optin behavior will be modified to
require that Google is configured as the default search provider. This CL
adds a new overload for StartPhysicalWebDiscovery so downstream users can
be modified to use the new signature that provides access to the browser
state, from which we can determine the default search provider.

BUG=630769

Committed: https://crrev.com/7d1e59e370b8810d5ba953d38887ed4a79af4a5e
Cr-Commit-Position: refs/heads/master@{#438003}
==========

[commit-bot: I haz the power, 2016-12-13 02:56:58]

Patchset 2 (id:??) landed as
https://crrev.com/7d1e59e370b8810d5ba953d38887ed4a79af4a5e
Cr-Commit-Position: refs/heads/master@{#438003}