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

chrome/browser/ui/browser.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_BROWSER_H_
 #define CHROME_BROWSER_UI_BROWSER_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 224 matching lines @@
   // Set indicator that this browser is being created via session restore.
   // This is used on the Mac (only) to determine animation style when the
   // browser window is shown.
   void set_is_session_restore(bool is_session_restore) {
     is_session_restore_ = is_session_restore;
   }
   bool is_session_restore() const {
     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_; }
   const std::string& app_name() const { return app_name_; }
   bool is_trusted_source() const { return is_trusted_source_; }
   Profile* profile() const { return profile_; }
   gfx::Rect override_bounds() const { return override_bounds_; }
   const std::string& initial_workspace() const { return initial_workspace_; }
 
   // |window()| will return NULL if called before |CreateBrowserWindow()|
@@ skipping 699 matching lines @@
   // The default values are typically loaded from local state (last session),
   // obtained from the last window of the same type, or obtained from the
   // shell shortcut's startup info.
   gfx::Rect override_bounds_;
   ui::WindowShowState initial_show_state_;
   const std::string initial_workspace_;
 
   // 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_;
 
   std::unique_ptr<ChromeBubbleManager> bubble_manager_;
 
   // The Find Bar. This may be NULL if there is no Find Bar, and if it is
   // non-NULL, it may or may not be visible.
   std::unique_ptr<FindBarController> find_bar_controller_;
 
   // Dialog box used for opening and saving files.
@@ skipping 44 matching lines @@
   // The following factory is used for chrome update coalescing.
   base::WeakPtrFactory<Browser> chrome_updater_factory_;
 
   // The following factory is used to close the frame at a later time.
   base::WeakPtrFactory<Browser> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(Browser);
 };
 
 #endif  // CHROME_BROWSER_UI_BROWSER_H_