[Autoplay] Add more rappor metrics for autoplay in cross-origin iframes

[Autoplay] Add more rappor metrics for autoplay in cross-origin iframes

This CL adds rappor metrics for recording the eTLD+1 of the top-level
frame and cross-origin iframe when the user pauses an autoplaying video.

BUG=672526
Committed: https://crrev.com/bdef306f2a6fbeea064ae552d063e4cee85dc5ec
Cr-Commit-Position: refs/heads/master@{#437976}

[Zhiqiang Zhang (Slow), 2016-12-08 18:21:19]

zqzhang@chromium.org changed reviewers:
+ mlamouri@chromium.org

[mlamouri (slow - plz ping), 2016-12-10 12:16:56]

lgtm but two high level comments: should we have histograms too? should we have
unit tests for AutoplayUmaHelper?

[Zhiqiang Zhang (Slow), 2016-12-12 15:52:55]

On 2016/12/10 12:16:56, mlamouri wrote:
> lgtm but two high level comments: should we have histograms too? should we
have
> unit tests for AutoplayUmaHelper?

For histograms, I think we should. Will add more in later patches.

For unit tests, currently we can do for the histograms since we have
"HistogramTester" utils in WebKit. For Rappor, I'm not sure if we can do it in
WebKit yet.

[Zhiqiang Zhang (Slow), 2016-12-12 15:53:35]

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

[Zhiqiang Zhang (Slow), 2016-12-12 15:53:35]

+holte for rappor.xml

[Steven Holte, 2016-12-12 18:12:52]

lgtm

[Zhiqiang Zhang (Slow), 2016-12-12 18:44:20]

The CQ bit was checked by zqzhang@chromium.org

[Zhiqiang Zhang (Slow), 2016-12-12 18:44:21]

The patchset sent to the CQ was uploaded after l-g-t-m from holte@chromium.org,
mlamouri@chromium.org
Link to the patchset: https://codereview.chromium.org/2563723002/#ps20001
(title: "rebased")

[commit-bot: I haz the power, 2016-12-12 18:45:07]

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-12-13 01:22:05]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481568260946890,
"parent_rev": "5268143b1f2b4765d3219b2b4a07d2f9bfaa9088", "commit_rev":
"e41a65e1cc139d03949b804ebdfed411e56bd086"}

[commit-bot: I haz the power, 2016-12-13 01:22:57]

Description was changed from

==========
[Autoplay] Add more rappor metrics for autoplay in cross-origin iframes

This CL adds rappor metrics for recording the eTLD+1 of the top-level
frame and cross-origin iframe when the user pauses an autoplaying video.

BUG=672526
==========

to

==========
[Autoplay] Add more rappor metrics for autoplay in cross-origin iframes

This CL adds rappor metrics for recording the eTLD+1 of the top-level
frame and cross-origin iframe when the user pauses an autoplaying video.

BUG=672526

Review-Url: https://codereview.chromium.org/2563723002
==========

[commit-bot: I haz the power, 2016-12-13 01:22:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-13 01:26:05]

Description was changed from

==========
[Autoplay] Add more rappor metrics for autoplay in cross-origin iframes

This CL adds rappor metrics for recording the eTLD+1 of the top-level
frame and cross-origin iframe when the user pauses an autoplaying video.

BUG=672526

Review-Url: https://codereview.chromium.org/2563723002
==========

to

==========
[Autoplay] Add more rappor metrics for autoplay in cross-origin iframes

This CL adds rappor metrics for recording the eTLD+1 of the top-level
frame and cross-origin iframe when the user pauses an autoplaying video.

BUG=672526

Committed: https://crrev.com/bdef306f2a6fbeea064ae552d063e4cee85dc5ec
Cr-Commit-Position: refs/heads/master@{#437976}
==========

[commit-bot: I haz the power, 2016-12-13 01:26:06]

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