Currently Variations headers are reported to Google properties for http and https requests. We will move to only include these headers on https requests.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a949b56566eb6519e7f78cd29a62705914ea82a8 commit a949b56566eb6519e7f78cd29a62705914ea82a8 Author: jwd <jwd@chromium.org> Date: Tue Dec 13 17:15:16 2016 Switch variations http headers to only be reported over https. BUG= 672950 Review-Url: https://codereview.chromium.org/2563103002 Cr-Commit-Position: refs/heads/master@{#438200} [modify] https://crrev.com/a949b56566eb6519e7f78cd29a62705914ea82a8/components/variations/net/variations_http_headers.cc [modify] https://crrev.com/a949b56566eb6519e7f78cd29a62705914ea82a8/components/variations/net/variations_http_headers.h [modify] https://crrev.com/a949b56566eb6519e7f78cd29a62705914ea82a8/components/variations/net/variations_http_headers_unittest.cc [modify] https://crrev.com/a949b56566eb6519e7f78cd29a62705914ea82a8/tools/metrics/histograms/histograms.xml
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a32d49767933c8642a9e4425b32f93bbbb7ef2c5 commit a32d49767933c8642a9e4425b32f93bbbb7ef2c5 Author: jwd <jwd@chromium.org> Date: Wed Dec 14 20:21:07 2016 Fixing histogram logging to better track variations headers https change. BUG= 672950 Review-Url: https://codereview.chromium.org/2575823003 Cr-Commit-Position: refs/heads/master@{#438605} [modify] https://crrev.com/a32d49767933c8642a9e4425b32f93bbbb7ef2c5/components/variations/net/variations_http_headers.cc [modify] https://crrev.com/a32d49767933c8642a9e4425b32f93bbbb7ef2c5/tools/metrics/histograms/histograms.xml
Your change meets the bar and is auto-approved for M56 (branch: 2924)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4e82827eb59ebfe3eb2003be3afa637383869587 commit 4e82827eb59ebfe3eb2003be3afa637383869587 Author: Jesse Doherty <jwd@chromium.org> Date: Fri Dec 16 18:39:48 2016 Switch variations http headers to only be reported over https. BUG= 672950 Review-Url: https://codereview.chromium.org/2563103002 Cr-Commit-Position: refs/heads/master@{#438200} (cherry picked from commit a949b56566eb6519e7f78cd29a62705914ea82a8) Review-Url: https://codereview.chromium.org/2587513002 . Cr-Commit-Position: refs/branch-heads/2924@{#528} Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059} [modify] https://crrev.com/4e82827eb59ebfe3eb2003be3afa637383869587/components/variations/net/variations_http_headers.cc [modify] https://crrev.com/4e82827eb59ebfe3eb2003be3afa637383869587/components/variations/net/variations_http_headers.h [modify] https://crrev.com/4e82827eb59ebfe3eb2003be3afa637383869587/components/variations/net/variations_http_headers_unittest.cc [modify] https://crrev.com/4e82827eb59ebfe3eb2003be3afa637383869587/tools/metrics/histograms/histograms.xml
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ee267be79291d49bb28a84232128c39b2f67ced2 commit ee267be79291d49bb28a84232128c39b2f67ced2 Author: Jesse Doherty <jwd@chromium.org> Date: Fri Dec 16 18:47:47 2016 Fixing histogram logging to better track variations headers https change. BUG= 672950 Review-Url: https://codereview.chromium.org/2575823003 Cr-Commit-Position: refs/heads/master@{#438605} (cherry picked from commit a32d49767933c8642a9e4425b32f93bbbb7ef2c5) Review-Url: https://codereview.chromium.org/2582723003 . Cr-Commit-Position: refs/branch-heads/2924@{#529} Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059} [modify] https://crrev.com/ee267be79291d49bb28a84232128c39b2f67ced2/components/variations/net/variations_http_headers.cc [modify] https://crrev.com/ee267be79291d49bb28a84232128c39b2f67ced2/tools/metrics/histograms/histograms.xml
Comment 1 by bugdroid1@chromium.org
, Dec 13 2016