More pre-MD odds and ends.

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

 // Copyright 2015 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.
 
 #include "chrome/browser/chromeos/login/signin/merge_session_throttling_utils.h"
 
 #include <stdint.h>
 
 #include "base/lazy_instance.h"
 #include "base/logging.h"
@@ skipping 117 matching lines @@
 
   switch (login_manager->state()) {
     case chromeos::OAuth2LoginManager::SESSION_RESTORE_NOT_STARTED:
       // The session restore for this profile hasn't even started yet. Don't
       // block for now.
       // In theory this should not happen since we should
       // kick off the session restore process for the newly added profile
       // before we attempt loading any page.
       if (user_manager::UserManager::Get()->IsLoggedInAsUserWithGaiaAccount() &&
           !user_manager::UserManager::Get()->IsLoggedInAsStub()) {
-        LOG(WARNING) << "Loading content for a profile without "
-                     << "session restore?";
+        DVLOG(1) << "Loading content for a profile without "

[sky, 2016/11/02 03:05:14]

Please ping the owner of this if they are ok with the cange. chromeos seems to
favor logging more in certain cases as otherwise they have no hope of tracking
down issues.

[Evan Stade, 2016/11/02 12:58:36]

+achuith@chromium.org, wdyt? I made this change because this warning was
spamming my console and making it hard to read test output. Rest of the file
seems to use DVLOG.

+                 << "session restore?";
       }
       return false;
     case chromeos::OAuth2LoginManager::SESSION_RESTORE_PREPARING:
     case chromeos::OAuth2LoginManager::SESSION_RESTORE_IN_PROGRESS: {
       // Check if the session restore has been going on for a while already.
       // If so, don't attempt to block page loading.
       if ((base::Time::Now() - login_manager->session_restore_start())
               .InSeconds() > kMaxSessionRestoreTimeInSec) {
         UnblockProfile(profile);
         return false;
@@ skipping 28 matching lines @@
 bool ShouldDelayUrl(const GURL& url) {
   // If we are loading google properties while merge session is in progress,
   // we will show delayed loading page instead.
   return !net::NetworkChangeNotifier::IsOffline() &&
          !AreAllSessionMergedAlready() &&
          google_util::IsGoogleHostname(url.host_piece(),
                                        google_util::ALLOW_SUBDOMAIN);
 }
 
 }  // namespace merge_session_throttling_utils