Get signals working in the EXTRA_DATA section of MHTML

chrome/browser/flag_descriptions.cc

Index: chrome/browser/flag_descriptions.cc
diff --git a/chrome/browser/flag_descriptions.cc b/chrome/browser/flag_descriptions.cc
index 63877eb6febaef5cfb98be3b192b88071265d96a..f3c22755528a7bf07434ce8aa07a3b305a927952 100644
--- a/chrome/browser/flag_descriptions.cc
+++ b/chrome/browser/flag_descriptions.cc
@@ -2030,6 +2030,14 @@ const char kOfflinePagesSvelteConcurrentLoadingDescription[] =
     "Android svelte (512MB RAM) devices. Otherwise, background loading will "
     "happen when the svelte device is idle.";
 
+const char kOfflinePagesLoadSignalCollectingName[] =
+    "Enables collecting load timing data for offline page snapshots.";

[Dmitry Titov, 2017/03/31 18:05:53]

Same note as in .h file.

I think we are not really after 'timing' signals because times in themselves say
little to us. For example, percentage of loaded resources says more... Trying to
avoid confusion for others reading the code. "telemetry" or "loading
completeness" are probably better than "timing".

[Pete Williamson, 2017/03/31 21:58:04]

Changed to "Enables load completeness data colllection " ...

+
+const char kOfflinePagesLoadSignalCollectingDescription[] =
+    "Enables load timing data collection while writing an offline page. "
+    "This data is collected in the snapshotted offline page to allow "
+    "data analysis to improve deciding when to make the offline snapshot.";
+
 const char kOfflinePagesPrefetchingName[] =
     "Enables suggested offline pages to be prefetched.";