Remove UWS Rappor

Remove UWS Rappor

TBR=laforge

BUG=696797
Review-Url: https://codereview.chromium.org/2731413003
Cr-Commit-Position: refs/heads/master@{#455300}
Committed: https://chromium.googlesource.com/chromium/src/+/35cb1d3e7a0b1ebe62b485ff06bf641a01d8c36a

[Steven Holte, 2017-03-07 19:47:53]

Description was changed from

==========
Remove UWS Rappor

BUG=
==========

to

==========
Remove UWS Rappor

BUG=696797
==========

[Steven Holte, 2017-03-07 19:48:35]

holte@chromium.org changed reviewers:
+ csharp@chromium.org

[csharp, 2017-03-07 21:55:20]

lgtm

[joenotcharles, 2017-03-07 22:11:40]

joenotcharles@google.com changed reviewers:
+ joenotcharles@google.com

[joenotcharles, 2017-03-07 22:11:41]

https://codereview.chromium.org/2731413003/diff/1/chrome/browser/safe_browsin...
File chrome/browser/safe_browsing/srt_fetcher_win.cc (left):

https://codereview.chromium.org/2731413003/diff/1/chrome/browser/safe_browsin...
chrome/browser/safe_browsing/srt_fetcher_win.cc:827:
SwReporterInvocation::BEHAVIOUR_LOG_TO_RAPPOR));
Can you remove BEHAVIOUR_LOG_TO_RAPPOR entirely please?

[Steven Holte, 2017-03-07 22:34:09]

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

[Steven Holte, 2017-03-07 22:34:10]

+nparker

https://codereview.chromium.org/2731413003/diff/1/chrome/browser/safe_browsin...
File chrome/browser/safe_browsing/srt_fetcher_win.cc (left):

https://codereview.chromium.org/2731413003/diff/1/chrome/browser/safe_browsin...
chrome/browser/safe_browsing/srt_fetcher_win.cc:827:
SwReporterInvocation::BEHAVIOUR_LOG_TO_RAPPOR));
On 2017/03/07 22:11:41, joenotcharles wrote:
> Can you remove BEHAVIOUR_LOG_TO_RAPPOR entirely please?

Done.

[Joe Mason, 2017-03-07 22:37:37]

joenotcharles@chromium.org changed reviewers:
+ joenotcharles@chromium.org

[Joe Mason, 2017-03-07 22:37:37]

lgtm

[Steven Holte, 2017-03-07 22:44:00]

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

[commit-bot: I haz the power, 2017-03-07 22:44:38]

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

[Nathan Parker, 2017-03-07 23:08:35]

safe_browsing/* LGTM

[commit-bot: I haz the power, 2017-03-07 23:36:22]

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

[commit-bot: I haz the power, 2017-03-07 23:36:23]

Dry run: This issue passed the CQ dry run.

[Steven Holte, 2017-03-07 23:50:14]

Description was changed from

==========
Remove UWS Rappor

BUG=696797
==========

to

==========
Remove UWS Rappor

TBR=laforge

BUG=696797
==========

[Steven Holte, 2017-03-07 23:50:14]

holte@chromium.org changed reviewers:
+ laforge@chromium.org

[Steven Holte, 2017-03-07 23:50:22]

The CQ bit was checked by holte@chromium.org

[Steven Holte, 2017-03-07 23:50:23]

The patchset sent to the CQ was uploaded after l-g-t-m from csharp@chromium.org
Link to the patchset: https://codereview.chromium.org/2731413003/#ps20001
(title: "Remove BEHAVIOUR_LOG_TO_RAPPOR")

[commit-bot: I haz the power, 2017-03-07 23:51: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, 2017-03-07 23:58:52]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488930622751110,
"parent_rev": "4121a1f03af37216d8f219cfdc4899e71ee2f4cf", "commit_rev":
"35cb1d3e7a0b1ebe62b485ff06bf641a01d8c36a"}

[commit-bot: I haz the power, 2017-03-07 23:59:33]

Description was changed from

==========
Remove UWS Rappor

TBR=laforge

BUG=696797
==========

to

==========
Remove UWS Rappor

TBR=laforge

BUG=696797

Review-Url: https://codereview.chromium.org/2731413003
Cr-Commit-Position: refs/heads/master@{#455300}
Committed:
https://chromium.googlesource.com/chromium/src/+/35cb1d3e7a0b1ebe62b485ff06bf...
==========

[commit-bot: I haz the power, 2017-03-07 23:59:35]

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