Tech Debt Repayment for StartupBrowserCreatorImpl Refactor

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 f232a48a600c663eb0fe17056a91e58d8d468028..05274b78dfa2176d74bc5ace0d5d7471361fe2f8 100644
--- a/chrome/browser/sessions/session_restore.h
+++ b/chrome/browser/sessions/session_restore.h
@@ -28,7 +28,11 @@ class WebContents;
 // variety is meant for startup and blocks until restore is complete.
 class SessionRestore {
  public:
-  enum Behavior {
+  // Bitmask representing behaviors available when restoring a session. Populate
+  // using the values below.
+  typedef uint32_t Behavior;

[sky, 2017/02/08 23:56:20]

typedef->usint
Also, to make this class name it BehaviorBitmask.

+
+  enum {

[sky, 2017/02/08 23:56:20]

enum class, which gives you the scoping should other enums be added to this
class.

     // Indicates the active tab of the supplied browser should be closed.
     CLOBBER_CURRENT_TAB          = 1 << 0,
 
@@ -56,7 +60,7 @@ class SessionRestore {
   // If |urls_to_open| is non-empty, a tab is added for each of the URLs.
   static Browser* RestoreSession(Profile* profile,
                                  Browser* browser,
-                                 uint32_t behavior,
+                                 Behavior behavior,
                                  const std::vector<GURL>& urls_to_open);
 
   // Restores the last session when the last session crashed. It's a wrapper