[Binding] [UMA] Add a UMA to track time spent in WindowProxy::initialize

Add a user metric to track time spent in WindowProxy::initialize
to track how efficiently we will reduce it.

BUG=325923, 588893
Committed: https://crrev.com/0a34f2d8be5cf7d5545de5fdc59057c320a9165e
Cr-Commit-Position: refs/heads/master@{#395810}

[peria, 2016-05-24 08:03:17]

Patchset #1 (id:1) has been deleted

[peria, 2016-05-24 08:05:17]

Description was changed from

==========
Add an UMA to measure time spent in WindowProxy::initialize

BUG=325923,588893
==========

to

==========
Add a user metric to measure time spent in WindowProxy::initialize
to track how efficiently we will reduce it.


BUG=325923,588893
==========

[peria, 2016-05-24 08:09:14]

peria@chromium.org changed reviewers:
+ kouhei@chromium.org

[peria, 2016-05-24 08:09:14]

kouhei@, could you check if it measures what we want before requesting OWNERs'
reviews?

[kouhei (in TOK), 2016-05-24 09:08:12]

https://codereview.chromium.org/2003293003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp (right):

https://codereview.chromium.org/2003293003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp:229: double startTime
= currentTimeMS();
use monotonicallyIncreasingTime() instead of currentTimeMS, or better, just use 
SCOPED_BLINK_UMA_HISTOGRAM_TIMER :)

[peria, 2016-05-24 09:36:23]

peria@chromium.org changed reviewers:
+ haraken@chromium.org, yukishiino@chromium.org

[peria, 2016-05-24 09:36:24]

PTL

https://codereview.chromium.org/2003293003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp (right):

https://codereview.chromium.org/2003293003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/bindings/core/v8/WindowProxy.cpp:229: double startTime
= currentTimeMS();
On 2016/05/24 09:08:12, kouhei wrote:
> use monotonicallyIncreasingTime() instead of currentTimeMS, or better, just
use 
> SCOPED_BLINK_UMA_HISTOGRAM_TIMER :)

Oh, I misunderstood the last argument of CustomCountHistogram.
Done.

[Yuki, 2016-05-24 10:09:34]

LGTM on my side, though I don't really know much about UMA.

[kouhei (in TOK), 2016-05-24 10:37:49]

lgtm

[peria, 2016-05-24 10:54:45]

Description was changed from

==========
Add a user metric to measure time spent in WindowProxy::initialize
to track how efficiently we will reduce it.


BUG=325923,588893
==========

to

==========
Add a user metric to track time spent in WindowProxy::initialize
to track how efficiently we will reduce it.


BUG=325923,588893
==========

[haraken, 2016-05-24 14:06:30]

LGTM

[peria, 2016-05-24 17:45:23]

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

[peria, 2016-05-24 17:45:23]

holte@, could you take a look as an OWNER of tools/metics?

[Steven Holte, 2016-05-24 18:11:34]

lgtm

[peria, 2016-05-25 00:40:44]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2016-05-25 00:41:28]

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

[commit-bot: I haz the power, 2016-05-25 05:50:07]

Description was changed from

==========
Add a user metric to track time spent in WindowProxy::initialize
to track how efficiently we will reduce it.


BUG=325923,588893
==========

to

==========
Add a user metric to track time spent in WindowProxy::initialize
to track how efficiently we will reduce it.


BUG=325923,588893
==========

[commit-bot: I haz the power, 2016-05-25 05:50:08]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-05-25 05:51:33]

Description was changed from

==========
Add a user metric to track time spent in WindowProxy::initialize
to track how efficiently we will reduce it.


BUG=325923,588893
==========

to

==========
Add a user metric to track time spent in WindowProxy::initialize
to track how efficiently we will reduce it.

BUG=325923,588893

Committed: https://crrev.com/0a34f2d8be5cf7d5545de5fdc59057c320a9165e
Cr-Commit-Position: refs/heads/master@{#395810}
==========

[commit-bot: I haz the power, 2016-05-25 05:51:34]

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