[ServiceWorkerCache] Cache and CacheStorage creation UMA

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 ac5fc8967af81924ff420da793841ec56866e75a..bdb2479b74535a1b5d447837257559340031bdfb 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -33540,6 +33540,23 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   </summary>
 </histogram>
 
+<histogram name="ServiceWorkerCache.CreateCacheStorageResult"
+    enum="BooleanCreated">
+  <owner>jkarlin@chromium.org</owner>
+  <summary>
+    The value of creating creating a new ServiceWorkerCacheStorage and its

[michaeln, 2015/03/13 22:47:16]

In the code, it looks like this is the result of creating an individual named
cache (ServiceWorkerCache) instance, yet this comment makes it sounnd like its
the result of creating the cache storage area for an origin
(ServiceWorkerCacheStorage).

also nit: creating creating

[Ilya Sherman, 2015/03/13 22:58:20]

nit: duplicate "creating"

[jkarlin, 2015/03/14 01:47:19]

Done.

[jkarlin, 2015/03/14 01:47:19]

I cleared up the naming confusion in the comment on
service_worker_cache_storage.cc

Got the nit, thanks.

+    directory. The value is Created (1) if the storage was able to be created.

[Ilya Sherman, 2015/03/13 22:58:20]

nit: I'd change this summary to something a little bit more like "Whether a new
ServiceWorkerCacheStorage and its directory were successfully created."  The
enum labels can speak for the rest.

[jkarlin, 2015/03/14 01:47:19]

Done.

+  </summary>
+</histogram>
+
+<histogram name="ServiceWorkerCache.InitBackendResult"
+    enum="ServiceWorkerCacheErrorType">
+  <owner>jkarlin@chromium.org</owner>
+  <summary>
+    The result of opening the backend in the ServiceWorker Cache API.
+  </summary>
+</histogram>
+
 <histogram name="Session.TotalDuration" units="milliseconds">
   <owner>mariakhomenko@chromium.org</owner>
   <owner>fqian@chromium.org</owner>
@@ -44795,6 +44812,11 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   <int value="1" label="Covered"/>
 </enum>
 
+<enum name="BooleanCreated" type="int">
+  <int value="0" label="Not created"/>
+  <int value="1" label="Created"/>
+</enum>
+
 <enum name="BooleanCredentialsLost" type="int">
   <int value="0" label="Found Credentials"/>
   <int value="1" label="Missing Credentials"/>
@@ -58354,6 +58376,13 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="11" label="SERVICE_UTILITY_FAILED_TO_START"/>
 </enum>
 
+<enum name="ServiceWorkerCacheErrorType" type="int">
+  <int value="0" label="OK"/>
+  <int value="1" label="Exists Error"/>
+  <int value="2" label="Storage Error"/>
+  <int value="3" label="Not Found Error"/>
+</enum>
+
 <enum name="ServiceWorkerDatabaseStatus" type="int">
   <int value="0" label="OK"/>
   <int value="1" label="Not Found Error"/>