Presentation API: add UMA.

Presentation API: add UMA.

BUG=507335
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200610

[mlamouri (slow - plz ping), 2015-08-12 16:56:53]

mlamouri@chromium.org changed reviewers:
+ avayvod@chromium.org, mfoltz@chromium.org

[mlamouri (slow - plz ping), 2015-08-12 16:56:53]

Could you PTAL and let me know if there are some UMA you think should be
added/removed?

[mark a. foltz, 2015-08-12 17:32:53]

lgtm

https://codereview.chromium.org/1290693003/diff/1/Source/modules/presentation...
File Source/modules/presentation/Presentation.idl (right):

https://codereview.chromium.org/1290693003/diff/1/Source/modules/presentation...
Source/modules/presentation/Presentation.idl:11:
[MeasureAs=PresentationDefaultRequest] attribute PresentationRequest?
defaultRequest;
Does this measure the frequency of setting this property?

[mlamouri (slow - plz ping), 2015-08-12 17:35:12]

On 2015/08/12 at 17:32:53, mfoltz wrote:
> lgtm
> 
>
https://codereview.chromium.org/1290693003/diff/1/Source/modules/presentation...
> File Source/modules/presentation/Presentation.idl (right):
> 
>
https://codereview.chromium.org/1290693003/diff/1/Source/modules/presentation...
> Source/modules/presentation/Presentation.idl:11:
[MeasureAs=PresentationDefaultRequest] attribute PresentationRequest?
defaultRequest;
> Does this measure the frequency of setting this property?

I believe this will measure read and write.

[whywhat, 2015-08-12 18:01:56]

The CQ bit was checked by avayvod@chromium.org

[whywhat, 2015-08-12 18:01:56]

lgtm

[commit-bot: I haz the power, 2015-08-12 18:02:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1290693003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1290693003/1

[commit-bot: I haz the power, 2015-08-12 18:10:08]

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

[commit-bot: I haz the power, 2015-08-12 18:10:09]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/3...)

[mlamouri (slow - plz ping), 2015-08-12 18:24:37]

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

[mlamouri (slow - plz ping), 2015-08-12 18:24:38]

Mike, can you review the UseCounter.h change?

[mlamouri (slow - plz ping), 2015-08-15 14:02:58]

Review ping: mkwst@, could you PTAL?

[Mike West, 2015-08-15 14:19:21]

UseCounter.h LGTM

[mlamouri (slow - plz ping), 2015-08-15 15:03:23]

The CQ bit was checked by mlamouri@chromium.org

[mlamouri (slow - plz ping), 2015-08-15 15:03:23]

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

[commit-bot: I haz the power, 2015-08-15 15:03:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1290693003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1290693003/20001

[commit-bot: I haz the power, 2015-08-15 16:32:16]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200610