Prevent the pre-session-restore log spam

chrome/browser/chromeos/login/signin/merge_session_throttling_utils.cc

Index: chrome/browser/chromeos/login/signin/merge_session_throttling_utils.cc
diff --git a/chrome/browser/chromeos/login/signin/merge_session_throttling_utils.cc b/chrome/browser/chromeos/login/signin/merge_session_throttling_utils.cc
index 182474e4d2c0a3631a72f1f86e6c2589fb782b6d..edab4bdd1a568f944ed3f5731660449dde17fbfd 100644
--- a/chrome/browser/chromeos/login/signin/merge_session_throttling_utils.cc
+++ b/chrome/browser/chromeos/login/signin/merge_session_throttling_utils.cc
@@ -184,4 +184,27 @@ bool ShouldDelayUrl(const GURL& url) {
                                        google_util::ALLOW_SUBDOMAIN);
 }
 
+bool IsSessionRestorePending(Profile* profile) {
+  if (!profile)
+    return false;

[Wez, 2016/12/08 18:33:26]

Is |false| the correct value in this case? What does it even mean to call this
for a null profile?

+
+  chromeos::OAuth2LoginManager* login_manager =
+      chromeos::OAuth2LoginManagerFactory::GetInstance()->GetForProfile(
+          profile);
+  bool pending_session_restore = false;
+  if (login_manager) {
+    switch (login_manager->state()) {

[Wez, 2016/12/08 18:33:26]

nit: This would be easier to read as an if(), rather than a switch.

+      case chromeos::OAuth2LoginManager::SESSION_RESTORE_PREPARING:
+      case chromeos::OAuth2LoginManager::SESSION_RESTORE_IN_PROGRESS:
+        pending_session_restore = true;
+        break;
+
+      default:
+        break;
+    }
+  }
+
+  return pending_session_restore;
+}
+
 }  // namespace merge_session_throttling_utils