Update some comments about variations headers API use.

Update some comments about variations headers API use.

Based on post-review comments on:
https://codereview.chromium.org/2558913003/

BUG=none
TBR=mattm@chromium.org,donnd@chromium.org,afakhry@chromium.org,noyau@chromium.org

Review-Url: https://codereview.chromium.org/2627033003
Cr-Commit-Position: refs/heads/master@{#443250}
Committed: https://chromium.googlesource.com/chromium/src/+/b2ea4c9c7a315e6da453842076f6ebee657bd371

[Alexei Svitkine (slow), 2017-01-11 22:33:23]

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

[Mathieu, 2017-01-11 22:33:56]

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

[Mathieu, 2017-01-11 22:33:57]

autofill lgtm

https://codereview.chromium.org/2627033003/diff/1/components/autofill/core/br...
File components/autofill/core/browser/autofill_download_manager.cc (right):

https://codereview.chromium.org/2627033003/diff/1/components/autofill/core/br...
components/autofill/core/browser/autofill_download_manager.cc:251: // not affect
transmission of experiments coming from the variations server..
nit: double .

[Mark P, 2017-01-11 22:34:58]

components/omnibox/... lgtm

[Alexei Svitkine (slow), 2017-01-11 22:36:50]

https://codereview.chromium.org/2627033003/diff/1/components/autofill/core/br...
File components/autofill/core/browser/autofill_download_manager.cc (right):

https://codereview.chromium.org/2627033003/diff/1/components/autofill/core/br...
components/autofill/core/browser/autofill_download_manager.cc:251: // not affect
transmission of experiments coming from the variations server..
On 2017/01/11 22:33:56, Mathieu Perreault wrote:
> nit: double .

Done.

[Alexei Svitkine (slow), 2017-01-12 15:19:59]

Description was changed from

==========
Update some comments about variations headers API use.

Based on post-review comments on:
https://codereview.chromium.org/2558913003/

BUG=none
==========

to

==========
Update some comments about variations headers API use.

Based on post-review comments on:
https://codereview.chromium.org/2558913003/

BUG=none
TBR=mattm@chromium.org,donnd@chromium.org,afakhry@chromium.org
==========

[Alexei Svitkine (slow), 2017-01-12 15:20:00]

asvitkine@chromium.org changed reviewers:
+ afakhry@chromium.org, donnd@chromium.org, mattm@chromium.org

[Alexei Svitkine (slow), 2017-01-12 15:20:19]

Description was changed from

==========
Update some comments about variations headers API use.

Based on post-review comments on:
https://codereview.chromium.org/2558913003/

BUG=none
TBR=mattm@chromium.org,donnd@chromium.org,afakhry@chromium.org
==========

to

==========
Update some comments about variations headers API use.

Based on post-review comments on:
https://codereview.chromium.org/2558913003/

BUG=none
TBR=mattm@chromium.org,donnd@chromium.org,afakhry@chromium.org,noyau@chromium...
==========

[Alexei Svitkine (slow), 2017-01-12 15:20:19]

asvitkine@chromium.org changed reviewers:
+ noyau@chromium.org

[Alexei Svitkine (slow), 2017-01-12 15:20:57]

TBR +=
mattm@chromium.org,donnd@chromium.org,afakhry@chromium.org,noyau@chromium.org

Since this is a downstream change of the variations request headers API.

[Alexei Svitkine (slow), 2017-01-12 15:21:04]

The CQ bit was checked by asvitkine@chromium.org

[Alexei Svitkine (slow), 2017-01-12 15:21:05]

The patchset sent to the CQ was uploaded after l-g-t-m from mathp@chromium.org,
mpearson@chromium.org
Link to the patchset: https://codereview.chromium.org/2627033003/#ps20001
(title: "Remove extra .")

[commit-bot: I haz the power, 2017-01-12 15:21:31]

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-01-12 16:13:26]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484234464555460,
"parent_rev": "92651104e939dbe94378a607e7acd381ca394fd3", "commit_rev":
"b2ea4c9c7a315e6da453842076f6ebee657bd371"}

[commit-bot: I haz the power, 2017-01-12 16:13:58]

Description was changed from

==========
Update some comments about variations headers API use.

Based on post-review comments on:
https://codereview.chromium.org/2558913003/

BUG=none
TBR=mattm@chromium.org,donnd@chromium.org,afakhry@chromium.org,noyau@chromium...
==========

to

==========
Update some comments about variations headers API use.

Based on post-review comments on:
https://codereview.chromium.org/2558913003/

BUG=none
TBR=mattm@chromium.org,donnd@chromium.org,afakhry@chromium.org,noyau@chromium...

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

[commit-bot: I haz the power, 2017-01-12 16:13:59]

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

[mattm, 2017-01-12 22:05:20]

safe_browsing lgtm