Gate sending referrers to CWS on whether the profile has SafeBrowsing enabled.

Gate sending referrers to CWS on whether the profile has SafeBrowsing enabled.

BUG=714129
Review-Url: https://codereview.chromium.org/2831263003
Cr-Commit-Position: refs/heads/master@{#466402}
Committed: https://chromium.googlesource.com/chromium/src/+/a603f32b1aa3556586f75909a2c73d3006b92edf

[robertshield, 2017-04-21 13:55:37]

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

[commit-bot: I haz the power, 2017-04-21 13:55:48]

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

[robertshield, 2017-04-21 13:58:00]

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

[commit-bot: I haz the power, 2017-04-21 14:45:55]

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

[commit-bot: I haz the power, 2017-04-21 14:45:57]

Dry run: This issue passed the CQ dry run.

[Devlin, 2017-04-21 17:09:43]

lgtm

https://codereview.chromium.org/2831263003/diff/1/chrome/browser/extensions/w...
File chrome/browser/extensions/webstore_inline_installer.cc (right):

https://codereview.chromium.org/2831263003/diff/1/chrome/browser/extensions/w...
chrome/browser/extensions/webstore_inline_installer.cc:115:
Profile::FromBrowserContext(web_contents()->GetBrowserContext());
nit: looks like WebstoreStandaloneInstaller has a profile() accessor that we
could use instead of this.

[robertshield, 2017-04-21 17:35:22]

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

[commit-bot: I haz the power, 2017-04-21 17:35:39]

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

[robertshield, 2017-04-21 18:02:54]

https://codereview.chromium.org/2831263003/diff/1/chrome/browser/extensions/w...
File chrome/browser/extensions/webstore_inline_installer.cc (right):

https://codereview.chromium.org/2831263003/diff/1/chrome/browser/extensions/w...
chrome/browser/extensions/webstore_inline_installer.cc:115:
Profile::FromBrowserContext(web_contents()->GetBrowserContext());
On 2017/04/21 17:09:43, Devlin wrote:
> nit: looks like WebstoreStandaloneInstaller has a profile() accessor that we
> could use instead of this.

Thanks, done.

[commit-bot: I haz the power, 2017-04-21 18:23:33]

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

[commit-bot: I haz the power, 2017-04-21 18:23:34]

Dry run: This issue passed the CQ dry run.

[robertshield, 2017-04-21 18:55:22]

The CQ bit was checked by robertshield@chromium.org

[robertshield, 2017-04-21 18:55:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
rdevlin.cronin@chromium.org
Link to the patchset: https://codereview.chromium.org/2831263003/#ps20001
(title: "Devlin feedback.")

[commit-bot: I haz the power, 2017-04-21 18:55:47]

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-04-21 18:59:47]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1492800922155130,
"parent_rev": "4162bafec82ea11c98d4dc2572337a039b79fdb9", "commit_rev":
"a603f32b1aa3556586f75909a2c73d3006b92edf"}

[commit-bot: I haz the power, 2017-04-21 18:59:57]

Description was changed from

==========
Gate sending referrers to CWS on whether the profile has SafeBrowsing enabled.

BUG=714129
==========

to

==========
Gate sending referrers to CWS on whether the profile has SafeBrowsing enabled.

BUG=714129

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

[commit-bot: I haz the power, 2017-04-21 18:59:59]

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