tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index bff5da0e325dbb4ac324563d48c8ad4799740da1..d911c57960ccf139cdbf94b3e670981ca4917e47 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -36498,6 +36498,32 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   </summary>
 </histogram>
 
+<histogram name="WebCore.Scripts.Deferred.TimeBetweenLoadedAndCompiled"
+    units="milliseconds">
+  <owner>marja@chromium.org</owner>
+  <summary>
+    Time between the events &quot;script is loaded&quot; and &quot;script is
+    compiled&quot; for deferred scripts.
+  </summary>
+</histogram>
+
+<histogram name="WebCore.Scripts.ParsingBlocking.AlreadyLoaded" enum="Boolean">

[Ilya Sherman, 2014/09/03 01:37:08]

nit: Please use a more custom-tailored boolean enum with labels "Blocked" and
"Unblocked" (or "Not blocked", if you prefer).

[marja, 2014/09/04 14:54:10]

Hmm, I'm not sure about the naming you suggest... so
"WebCore.Scripts.ParsingBlocking.AlreadyLoaded" = "Blocked" would mean that the
script was not already loaded, and
"WebCore.Scripts.ParsingBlocking.AlreadyLoaded" = "Not blocked / Unblocked" 
would mean that the script *was* already loaded? IMO this is confusing.

To clarify: "ParsingBlocking" is here because this monitors attributes of
"parsing blocking scripts". It does not monitor whether something was parsing
blocking or not; it monitors whether a parsing blocking script was already
loaded from the network when it was needed. Did you get the term "Blocked" from
"ParserBlocking"?

How about adding BooleanLoaded with "Loaded" / "Not loaded", and then
"WebCore.Scripts.ParsingBlocking.AlreadyLoaded" = "Loaded" would mean that the
script was loaded...

Otoh, "WebCore.Scripts.ParsingBlocking.AlreadyLoaded" = True expresses the same
thing. So IDK.

[Ilya Sherman, 2014/09/04 17:16:40]

Sorry, I meant "Loaded" / "Not loaded" -- not sure why I wrote "blocked" / "not
blocked".  It's not critical to add this enum, but IMO it's slightly nicer when
using the dashboard to have semantically rich labels so that I don't need to
glance back at the histogram name or <summary> to remind myself which case is
"true" and which case is "false".  LGTM either way.

+  <owner>marja@chromium.org</owner>
+  <summary>
+    Whether a parsing blocking script was already preloaded from the net or the
+    cache by the time it was needed.
+  </summary>
+</histogram>
+
+<histogram name="WebCore.Scripts.ParsingBlocking.TimeBetweenLoadedAndCompiled"
+    units="milliseconds">
+  <owner>marja@chromium.org</owner>
+  <summary>
+    Time between the events &quot;script is loaded&quot; and &quot;script is
+    compiled&quot; for parsing blocking scripts.
+  </summary>
+</histogram>
+
 <histogram name="WebCore.V8DOMWindowShell.createContext.IsolatedWorld"
     units="milliseconds">
   <obsolete>