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 6625d3e4ef6aee9665ea51956bd7c5c08330c559..200deb59ad3138c358b01591131d3f722144a63a 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -34677,6 +34677,38 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="OfflinePages.ExpirePage.BatchSize">
+  <owner>romax@chromium.org</owner>
+  <summary>Number of pages that are expired in a batch.</summary>
+</histogram>
+
+<histogram name="OfflinePages.ExpirePage.PageLifetime" units="minutes">
+  <owner>romax@chromium.org</owner>
+  <summary>
+    Length of time between when an offline pages was created and was expired.
+  </summary>
+</histogram>
+
+<histogram name="OfflinePages.ExpirePage.Result"

[jianli, 2016/05/27 19:16:13]

Suggest to name this as OfflinePages.ClearStorageResult since StorageManager
might not be just used for expiring pages.

[romax, 2016/05/27 20:36:02]

Done.

+    enum="OfflinePagesClearStorageResult">
+  <owner>romax@chromium.org</owner>
+  <summary>Result of asking storage manager to clear storage.</summary>
+</histogram>
+
+<histogram name="OfflinePages.ExpirePage.StoreUpdateResult"
+    enum="BooleanSuccess">
+  <owner>romax@chromium.org</owner>
+  <summary>Result of updating expired page in store.</summary>
+</histogram>
+
+<histogram name="OfflinePages.ExpirePage.TimeSinceLastAccess" units="minutes">
+  <owner>romax@chromium.org</owner>
+  <summary>
+    Length of time between when an offline pages was last opened and was
+    expired.
+  </summary>
+</histogram>
+
 <histogram name="OfflinePages.Filter.OnlineWhenEntering" enum="Boolean">
   <obsolete>
     Deprecated 5/2016. Offline pages no longer depend on bookmarks UI.
@@ -81344,6 +81376,14 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="2" label="Store reload failed"/>
 </enum>
 
+<enum name="OfflinePagesClearStorageResult" type="int">
+  <int value="0" label="Success"/>
+  <int value="1" label="Unnecessary to clean"/>
+  <int value="2" label="Page expiration failed"/>
+  <int value="3" label="Page record deletion failed"/>
+  <int value="4" label="Both expiration and deletion failed"/>
+</enum>
+
 <enum name="OfflinePagesDeletePageResult" type="int">
   <int value="0" label="Success"/>
   <int value="1" label="Cancelled"/>
@@ -94536,6 +94576,8 @@ To add a new entry, add it with any value and run test to compute valid value.
   <affected-histogram name="OfflinePages.DeletePage.LastOpenToCreated"/>
   <affected-histogram name="OfflinePages.DeletePage.PageSize"/>
   <affected-histogram name="OfflinePages.DeletePage.TimeSinceLastOpen"/>
+  <affected-histogram name="OfflinePages.ExpirePage.PageLifetime"/>
+  <affected-histogram name="OfflinePages.ExpirePage.TimeSinceLastAccess"/>
   <affected-histogram name="OfflinePages.FirstOpenSinceCreated"/>
   <affected-histogram name="OfflinePages.OpenSinceLastOpen"/>
   <affected-histogram name="OfflinePages.PageLifetime"/>