Add UMA to understand the typical total size of ServiceWorker script [1/2 blink]

Add UMA to understand the typical total size of ServiceWorker script [1/2 blink]

This cl introduces three UMAs.
- ServiceWorker.ScriptCount
- ServiceWorker.ScriptTotalSize
- ServiceWorker.ScriptCachedMetadataTotalSize


1/2 blink: This cl.
2/2 chromium: https://codereview.chromium.org/1055423002/

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

[horo, 2015-04-03 08:33:58]

horo@chromium.org changed reviewers:
+ ksakamoto@chromium.org

[horo, 2015-04-03 08:33:58]

ksakamoto@
Could you please review this?

[Kunihiko Sakamoto, 2015-04-03 09:10:00]

lgtm

[horo, 2015-04-03 09:12:40]

horo@chromium.org changed reviewers:
+ haraken@chromium.org

[horo, 2015-04-03 09:12:41]

haraken@
Could you please review this?

[haraken, 2015-04-03 09:16:53]

LGTM

https://codereview.chromium.org/1059193002/diff/1/Source/modules/serviceworke...
File Source/modules/serviceworkers/ServiceWorkerGlobalScope.cpp (right):

https://codereview.chromium.org/1059193002/diff/1/Source/modules/serviceworke...
Source/modules/serviceworkers/ServiceWorkerGlobalScope.cpp:110:
Platform::current()->histogramCustomCounts("ServiceWorker.ScriptCount",
m_scriptCount, 1, 1000, 50);

Do we probably need if(Platform::current()) check here? Platform::current()
sometimes doesn't exist in unittests etc. (This is a bug.)

[horo, 2015-04-03 09:27:01]

Thank you!

https://codereview.chromium.org/1059193002/diff/1/Source/modules/serviceworke...
File Source/modules/serviceworkers/ServiceWorkerGlobalScope.cpp (right):

https://codereview.chromium.org/1059193002/diff/1/Source/modules/serviceworke...
Source/modules/serviceworkers/ServiceWorkerGlobalScope.cpp:110:
Platform::current()->histogramCustomCounts("ServiceWorker.ScriptCount",
m_scriptCount, 1, 1000, 50);
On 2015/04/03 09:16:53, haraken wrote:
> 
> Do we probably need if(Platform::current()) check here? Platform::current()
> sometimes doesn't exist in unittests etc. (This is a bug.)

Done.

[horo, 2015-04-03 10:42:20]

The CQ bit was checked by horo@chromium.org

[horo, 2015-04-03 10:42:20]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, ksakamoto@chromium.org
Link to the patchset: https://codereview.chromium.org/1059193002/#ps20001
(title: "check Platform::current()")

[commit-bot: I haz the power, 2015-04-03 10:42:34]

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

[commit-bot: I haz the power, 2015-04-03 11:06:57]

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