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

chrome/browser/ui/browser.h

Index: chrome/browser/ui/browser.h
diff --git a/chrome/browser/ui/browser.h b/chrome/browser/ui/browser.h
index 5940ad9fbc27f456e15323ed83aac47c7852197e..800c8187a1f117b429ef06431a0b20380ae5cc54 100644
--- a/chrome/browser/ui/browser.h
+++ b/chrome/browser/ui/browser.h
@@ -242,6 +242,14 @@ class Browser : public TabStripModelObserver,
     return is_session_restore_;
   }
 
+  void set_is_used_for_session_restore(bool is_used_for_session_restore) {
+    is_used_for_session_restore_ = is_used_for_session_restore;

[sky, 2017/07/21 14:48:09]

How does this differ from is_session_restore_ above?

[ducbui, 2017/07/21 15:19:58]

The is_session_restore_ above is used on Mac only to indicate that session
restore is in progress because when Mac window shows it inhibits animation
during session restore. It was added in this CL:
https://codereview.chromium.org/7809013

The new is_used_for_session_restore_ indicates not only that a new browser is
created for session restore but also an existing browser is being used to hold
restored tabs. I want to track foreground tabs created by session restore and
adding this property to browser is sufficient.

+  }
+
+  bool is_used_for_session_restore() const {
+    return is_used_for_session_restore_;
+  }
+
   // Accessors ////////////////////////////////////////////////////////////////
 
   Type type() const { return type_; }
@@ -961,6 +969,9 @@ class Browser : public TabStripModelObserver,
   // Tracks when this browser is being created by session restore.
   bool is_session_restore_;
 
+  // True if a newly created or exisiting browser is used for session restore.
+  bool is_used_for_session_restore_;
+
   std::unique_ptr<chrome::UnloadController> unload_controller_;
   std::unique_ptr<chrome::FastUnloadController> fast_unload_controller_;