Update links for the SBER opt-in on interstitials

Update links for the SBER opt-in on interstitials

BUG=639166
TBR=mpearson@chromium.org

Committed: https://crrev.com/4f76c4dddda7ee81f70cbc3b61191ce1188abd7e
Cr-Commit-Position: refs/heads/master@{#418146}

[felt, 2016-09-12 17:45:46]

Patchset #1 (id:1) has been deleted

[felt, 2016-09-12 17:45:56]

Patchset #1 (id:20001) has been deleted

[felt, 2016-09-12 17:46:06]

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

[commit-bot: I haz the power, 2016-09-12 17:46:43]

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

[felt, 2016-09-12 17:47:11]

Description was changed from

==========
Update links for the SBER opt-in on interstitials

BUG=639166
==========

to

==========
Update links for the SBER opt-in on interstitials

BUG=639166
==========

[felt, 2016-09-12 17:47:11]

felt@chromium.org changed reviewers:
+ nparker@chromium.org

[felt, 2016-09-12 17:47:24]

nparker, PTAL? this was a request from privacy team

[commit-bot: I haz the power, 2016-09-12 19:14:31]

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

[commit-bot: I haz the power, 2016-09-12 19:14:32]

Dry run: This issue passed the CQ dry run.

[Nathan Parker, 2016-09-13 01:06:42]

lgtm

https://codereview.chromium.org/2330753002/diff/40001/components/security_int...
File components/security_interstitials/core/controller_client.cc (right):

https://codereview.chromium.org/2330753002/diff/40001/components/security_int...
components/security_interstitials/core/controller_client.cc:55: GURL
privacy_url(
How about "whitepaper_url" for the var name?

[felt, 2016-09-13 01:45:11]

https://codereview.chromium.org/2330753002/diff/40001/components/security_int...
File components/security_interstitials/core/controller_client.cc (right):

https://codereview.chromium.org/2330753002/diff/40001/components/security_int...
components/security_interstitials/core/controller_client.cc:55: GURL
privacy_url(
On 2016/09/13 01:06:42, Nathan Parker wrote:
> How about "whitepaper_url" for the var name?

Done.

[felt, 2016-09-13 01:45:54]

Description was changed from

==========
Update links for the SBER opt-in on interstitials

BUG=639166
==========

to

==========
Update links for the SBER opt-in on interstitials

BUG=639166
TBR=mpearson@chromium.org
==========

[felt, 2016-09-13 01:45:54]

felt@chromium.org changed reviewers:
+ mpearson@chromium.org

[felt, 2016-09-13 01:46:07]

TBR-ing mpearson for trivial histograms.xml change

[felt, 2016-09-13 01:46:19]

The CQ bit was checked by felt@chromium.org

[felt, 2016-09-13 01:46:19]

The patchset sent to the CQ was uploaded after l-g-t-m from nparker@chromium.org
Link to the patchset: https://codereview.chromium.org/2330753002/#ps60001
(title: "Update variable name")

[commit-bot: I haz the power, 2016-09-13 01:46:36]

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-09-13 02:50:02]

Description was changed from

==========
Update links for the SBER opt-in on interstitials

BUG=639166
TBR=mpearson@chromium.org
==========

to

==========
Update links for the SBER opt-in on interstitials

BUG=639166
TBR=mpearson@chromium.org
==========

[commit-bot: I haz the power, 2016-09-13 02:50:02]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-09-13 02:51:38]

Description was changed from

==========
Update links for the SBER opt-in on interstitials

BUG=639166
TBR=mpearson@chromium.org
==========

to

==========
Update links for the SBER opt-in on interstitials

BUG=639166
TBR=mpearson@chromium.org

Committed: https://crrev.com/4f76c4dddda7ee81f70cbc3b61191ce1188abd7e
Cr-Commit-Position: refs/heads/master@{#418146}
==========

[commit-bot: I haz the power, 2016-09-13 02:51:39]

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

[Mark P, 2016-09-13 04:12:13]

official histograms.xml lgtm

On 2016/09/13 01:46:07, felt wrote:
> TBR-ing mpearson for trivial histograms.xml change