[Tab Metrics] Measure FP, FCP and FMP for Foreground Tab during Session Restore

chrome/browser/sessions/session_restore.h

Index: chrome/browser/sessions/session_restore.h
diff --git a/chrome/browser/sessions/session_restore.h b/chrome/browser/sessions/session_restore.h
index 77fa7c0a8cd72c2c8e97a28501e9ad811d5f8d87..82e7095c8d8774a1938d8865354f581574d30481 100644
--- a/chrome/browser/sessions/session_restore.h
+++ b/chrome/browser/sessions/session_restore.h
@@ -122,6 +122,18 @@ class SessionRestore {
                            ConcurrentSessionRestores);
   FRIEND_TEST_ALL_PREFIXES(SessionRestoreObserverTest,
                            TabManagerShouldObserveSessionRestore);
+  FRIEND_TEST_ALL_PREFIXES(SessionRestorePageLoadMetricsObserverTest,

[Bryan McQuade, 2017/08/09 15:41:35]

eek, it's slightly concerning to me that we need to call private methods from
tests - it's usually a hint that we may not be testing the real production
behavior directly. is this standard practice for session restore tests? i don't
know the code well enough to make a firm call here, but this is slightly
worrying to me.

[ducbui, 2017/08/09 19:43:07]

I will remove these because OnWillRestoreTab() is made public in a
related CL (https://chromium-review.googlesource.com/c/601545).

+                           RestoreSingleForegroundTab);
+  FRIEND_TEST_ALL_PREFIXES(SessionRestorePageLoadMetricsObserverTest,
+                           RestoreMultipleForegroundTabs);
+  FRIEND_TEST_ALL_PREFIXES(SessionRestorePageLoadMetricsObserverTest,
+                           RestoreBackgroundTab);
+  FRIEND_TEST_ALL_PREFIXES(SessionRestorePageLoadMetricsObserverTest,
+                           HideTabBeforeFirstPaints);
+  FRIEND_TEST_ALL_PREFIXES(SessionRestorePageLoadMetricsObserverTest,
+                           SwitchInitialRestoredForegroundTab);
+  FRIEND_TEST_ALL_PREFIXES(SessionRestorePageLoadMetricsObserverTest,
+                           MultipleSessionRestores);
 
   // Session restore observer list.
   using SessionRestoreObserverList = base::ObserverList<SessionRestoreObserver>;