[Tab Metrics] Added TabManager.SessionRestore.SwitchToTab UMA.

chrome/browser/resource_coordinator/tab_manager_web_contents_data.h

Index: chrome/browser/resource_coordinator/tab_manager_web_contents_data.h
diff --git a/chrome/browser/resource_coordinator/tab_manager_web_contents_data.h b/chrome/browser/resource_coordinator/tab_manager_web_contents_data.h
index b77e5cb8388a915c2097feaa31c4099f0cd876ac..6f1f2e7384db94115b6c22432790928389aa9cca 100644
--- a/chrome/browser/resource_coordinator/tab_manager_web_contents_data.h
+++ b/chrome/browser/resource_coordinator/tab_manager_web_contents_data.h
@@ -29,11 +29,10 @@ namespace resource_coordinator {
 // occurs in the main frame, which happens if the user navigates to a new page
 // and the WebContents is reused.
 //
+// These values are used in the TabManager.SessionRestore.SwitchToTab UMA.
+//
 // TODO(shaseley): *switch to the new done signal (network and cpu quiescence)
 // when available.
-//
-// TODO(shaseley): This will become an UMA histogram once the TabManager is
-// aware of session restore begin/end and background tab loading begin/end.
 enum TabLoadingState {

[Mark P, 2017/06/29 18:25:51]

This needs the standard warning about changing values.
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histo...

[shaseley, 2017/06/29 19:45:16]

Done.

   TAB_IS_NOT_LOADING = 0,
   TAB_IS_LOADING = 1,
@@ -129,6 +128,7 @@ class TabManager::WebContentsData
   // Needed to access tab_data_.
   FRIEND_TEST_ALL_PREFIXES(TabManagerWebContentsDataTest, CopyState);
   FRIEND_TEST_ALL_PREFIXES(TabManagerWebContentsDataTest, TabLoadingState);
+  FRIEND_TEST_ALL_PREFIXES(TabManagerTest, HistogramsSessionRestoreSwitchToTab);
 
   struct Data {
     Data();