Count usage of webkitRequestFullscreen in shadow

Count usage of webkitRequestFullscreen in shadow

This counts webkitRequestFullscreen API usage in a shadow tree,
to see how popular requesting fullscreen of an element
in a shadow tree.

Overall usage is already counted as
"PrefixedRequestFullscreen" and "PrefixedRequestFullScreen".

BUG=535164
Committed: https://crrev.com/ab8c38147b4e094e70f1b627a46111ee5135db62
Cr-Commit-Position: refs/heads/master@{#403105}

[kochi, 2016-06-29 10:45:48]

Description was changed from

==========
Count usage of webkitRequestFullscreen in shadow

This counts webkitRequestFullscreen API usage in shadow tree.

BUG=535164
==========

to

==========
Count usage of webkitRequestFullscreen in shadow

This counts webkitRequestFullscreen API usage in shadow tree.

BUG=535164
==========

[kochi, 2016-06-29 10:45:48]

kochi@chromium.org changed reviewers:
+ hayato@chromium.org, isherman@chromium.org

[kochi, 2016-06-29 10:51:09]

Patchset #1 (id:1) has been deleted

[kochi, 2016-06-29 10:51:56]

PTAL hayato-san, could you review this?

isherman@, could you review for histograms.xml?

Note: At this point, unprefixed API is test-only and not
counted at all even without shadow.

[hayato, 2016-06-29 11:02:24]

Could you explain briefly why we want to count only for a shadow tree?

[kochi, 2016-06-29 11:06:09]

Description was changed from

==========
Count usage of webkitRequestFullscreen in shadow

This counts webkitRequestFullscreen API usage in shadow tree.

BUG=535164
==========

to

==========
Count usage of webkitRequestFullscreen in shadow

This counts webkitRequestFullscreen API usage in a shadow tree,
to see how popular requesting fullscreen of an element
in a shadow tree.

Overall usage is already counted as
"PrefixedRequestFullscreen" and "PrefixedRequestFullScreen".

BUG=535164
==========

[kochi, 2016-06-29 11:06:23]

Added description.

[kochi, 2016-06-29 11:19:11]

On 2016/06/29 11:06:23, kochi wrote:
> Added description.

FYI I did the similar thing for Element.requestPointerLock()
https://chromium.googlesource.com/chromium/src.git/+/d148a25d4b552d08fe0971fa...
(ElementRequestPointerLockInShadow)

[Ilya Sherman, 2016-06-29 20:32:30]

histograms.xml lgtm

[tkent, 2016-06-29 22:55:20]

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

[tkent, 2016-06-29 22:55:21]

lgtm

[kochi, 2016-06-30 01:56:41]

The CQ bit was checked by kochi@chromium.org

[commit-bot: I haz the power, 2016-06-30 01:57: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-06-30 01:58:45]

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

[commit-bot: I haz the power, 2016-06-30 01:58:46]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[kochi, 2016-06-30 02:13:49]

The CQ bit was checked by kochi@chromium.org

[kochi, 2016-06-30 02:13:49]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org,
isherman@chromium.org
Link to the patchset: https://codereview.chromium.org/2109113002/#ps40001
(title: "rebase, fix compilation")

[commit-bot: I haz the power, 2016-06-30 02:14: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, 2016-06-30 03:55:21]

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

[commit-bot: I haz the power, 2016-06-30 03:55:23]

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

[hayato, 2016-06-30 03:57:31]

lgtm

[kochi, 2016-06-30 05:29:01]

The CQ bit was checked by kochi@chromium.org

[commit-bot: I haz the power, 2016-06-30 05:29:33]

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-06-30 06:54:56]

Description was changed from

==========
Count usage of webkitRequestFullscreen in shadow

This counts webkitRequestFullscreen API usage in a shadow tree,
to see how popular requesting fullscreen of an element
in a shadow tree.

Overall usage is already counted as
"PrefixedRequestFullscreen" and "PrefixedRequestFullScreen".

BUG=535164
==========

to

==========
Count usage of webkitRequestFullscreen in shadow

This counts webkitRequestFullscreen API usage in a shadow tree,
to see how popular requesting fullscreen of an element
in a shadow tree.

Overall usage is already counted as
"PrefixedRequestFullscreen" and "PrefixedRequestFullScreen".

BUG=535164
==========

[commit-bot: I haz the power, 2016-06-30 06:54:57]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-06-30 06:55:01]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-30 06:57:54]

Description was changed from

==========
Count usage of webkitRequestFullscreen in shadow

This counts webkitRequestFullscreen API usage in a shadow tree,
to see how popular requesting fullscreen of an element
in a shadow tree.

Overall usage is already counted as
"PrefixedRequestFullscreen" and "PrefixedRequestFullScreen".

BUG=535164
==========

to

==========
Count usage of webkitRequestFullscreen in shadow

This counts webkitRequestFullscreen API usage in a shadow tree,
to see how popular requesting fullscreen of an element
in a shadow tree.

Overall usage is already counted as
"PrefixedRequestFullscreen" and "PrefixedRequestFullScreen".

BUG=535164

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

[commit-bot: I haz the power, 2016-06-30 06:57:56]

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