Fix AutoplayUmaHelper when autoplay is initiated from multiple sources

Fix AutoplayUmaHelper when autoplay is initiated from multiple sources

This CL makes AutoplayUmaHelper to handle the case where the page
autoplays by both attribute and play() method. Therefore we can avoid
counting the metrics into the wrong bucket.

BUG=699176
Review-Url: https://codereview.chromium.org/2767093002
Cr-Commit-Position: refs/heads/master@{#459215}
Committed: https://chromium.googlesource.com/chromium/src/+/3d3dfcd34e38fcecaa1ff14bcd12cbc98259f36c

[Zhiqiang Zhang (Slow), 2017-03-22 17:26:22]

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

[whywhat, 2017-03-22 21:13:19]

lgtm

[Zhiqiang Zhang (Slow), 2017-03-22 21:14:45]

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

[Zhiqiang Zhang (Slow), 2017-03-22 21:14:46]

+jwd: histograms.xml

[jwd, 2017-03-23 14:38:56]

lgtm

[Zhiqiang Zhang (Slow), 2017-03-23 15:12:46]

+mlamouri FYI

This CL should not change much, but we should be counting in the right bucket.

[Zhiqiang Zhang (Slow), 2017-03-23 15:15:48]

The CQ bit was checked by zqzhang@chromium.org

[Zhiqiang Zhang (Slow), 2017-03-23 15:15:49]

The patchset sent to the CQ was uploaded after l-g-t-m from
avayvod@chromium.org, jwd@chromium.org
Link to the patchset: https://codereview.chromium.org/2767093002/#ps20001
(title: "fixed rappor")

[commit-bot: I haz the power, 2017-03-23 15:16:14]

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-23 15:26:08]

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

[commit-bot: I haz the power, 2017-03-23 15:26:09]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Zhiqiang Zhang (Slow), 2017-03-23 17:20:14]

The CQ bit was checked by zqzhang@chromium.org

[Zhiqiang Zhang (Slow), 2017-03-23 17:20:15]

The patchset sent to the CQ was uploaded after l-g-t-m from
avayvod@chromium.org, jwd@chromium.org
Link to the patchset: https://codereview.chromium.org/2767093002/#ps40001
(title: "fixed build")

[commit-bot: I haz the power, 2017-03-23 17:20:37]

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-23 17:36:05]

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

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

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[Zhiqiang Zhang (Slow), 2017-03-23 18:30:18]

The CQ bit was checked by zqzhang@chromium.org

[commit-bot: I haz the power, 2017-03-23 18:31:02]

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-23 18:58:30]

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

[commit-bot: I haz the power, 2017-03-23 18:58:31]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Zhiqiang Zhang (Slow), 2017-03-23 19:13:11]

The CQ bit was checked by zqzhang@chromium.org

[Zhiqiang Zhang (Slow), 2017-03-23 19:13:11]

The patchset sent to the CQ was uploaded after l-g-t-m from
avayvod@chromium.org, jwd@chromium.org
Link to the patchset: https://codereview.chromium.org/2767093002/#ps60001
(title: "fixed test")

[commit-bot: I haz the power, 2017-03-23 19:13:57]

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-23 21:07:07]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490296391566840,
"parent_rev": "942ef62b8eb39ebd5ca23b0bce07bc0297e20499", "commit_rev":
"3d3dfcd34e38fcecaa1ff14bcd12cbc98259f36c"}

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

Description was changed from

==========
Fix AutoplayUmaHelper when autoplay is initiated from multiple sources

This CL makes AutoplayUmaHelper to handle the case where the page
autoplays by both attribute and play() method. Therefore we can avoid
counting the metrics into the wrong bucket.

BUG=699176
==========

to

==========
Fix AutoplayUmaHelper when autoplay is initiated from multiple sources

This CL makes AutoplayUmaHelper to handle the case where the page
autoplays by both attribute and play() method. Therefore we can avoid
counting the metrics into the wrong bucket.

BUG=699176

Review-Url: https://codereview.chromium.org/2767093002
Cr-Commit-Position: refs/heads/master@{#459215}
Committed:
https://chromium.googlesource.com/chromium/src/+/3d3dfcd34e38fcecaa1ff14bcd12...
==========

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

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/3d3dfcd34e38fcecaa1ff14bcd12...