From 9a127f2a2b70dca7271729b751d189f22e7ca3cf Mon Sep 17 00:00:00 2001
From: "tim@chromium.org"
 <tim@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Date: Fri, 17 Sep 2010 21:05:16 +0000
Subject: Remove unnecessary log spew from ProfileSyncService.

BUG=none.
TEST=examine logs from browser/ui/unit/interactive_ui tests, profile_sync_service should not be in top 20 spammers.

Review URL: http://codereview.chromium.org/3436017

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59848 0039d316-1c4b-4281-b951-d872f2087c98
---
 chrome/browser/sync/profile_sync_service.cc   | 12 ------------
 chrome/browser/sync/profile_sync_service.h    |  2 ++
 chrome/browser/sync/resources/about_sync.html |  6 ++++++
 3 files changed, 8 insertions(+), 12 deletions(-)

(limited to 'chrome/browser/sync')

diff --git a/chrome/browser/sync/profile_sync_service.cc b/chrome/browser/sync/profile_sync_service.cc
index c344b83..277eaed 100644
--- a/chrome/browser/sync/profile_sync_service.cc
+++ b/chrome/browser/sync/profile_sync_service.cc
@@ -90,13 +90,8 @@ ProfileSyncService::ProfileSyncService(ProfileSyncFactory* factory,
   // If this is an official build, it will always be one of the above.
   std::string channel = platform_util::GetVersionStringModifier();
   if (channel.empty() || channel == "beta") {
-    LOG(INFO) << "Detected official build, using official sync server.";
     sync_service_url_ = GURL(kSyncServerUrl);
-  } else {
-    LOG(INFO) << "Detected official build, but using dev channel sync server.";
   }
-#else
-  LOG(INFO) << "Unofficial build, using dev channel sync server.";
 #endif
 }
 
@@ -149,7 +144,6 @@ void ProfileSyncService::LoadMigratedCredentials(const std::string& username,
 }
 
 void ProfileSyncService::Initialize() {
-  LOG(INFO) << "Starting ProfileSyncService.";
   InitSettings();
   RegisterPreferences();
 
@@ -256,8 +250,6 @@ void ProfileSyncService::InitSettings() {
     }
   }
 
-  LOG(INFO) << "Using " << sync_service_url_ << " for sync server URL.";
-
   // Override the notification server host from the command-line, if provided.
   if (command_line.HasSwitch(switches::kSyncNotificationHost)) {
     std::string value(command_line.GetSwitchValueASCII(
@@ -386,8 +378,6 @@ void ProfileSyncService::StartUp() {
 
   DCHECK(AreCredentialsAvailable());
 
-  LOG(INFO) << "ProfileSyncService bringing up backend host.";
-
   last_synced_time_ = base::Time::FromInternalValue(
       profile_->GetPrefs()->GetInt64(prefs::kSyncLastSyncedTime));
 
@@ -436,8 +426,6 @@ void ProfileSyncService::ClearServerData() {
 }
 
 void ProfileSyncService::DisableForUser() {
-  LOG(INFO) << "Disabling sync for user.";
-
   // Clear prefs (including SyncSetupHasCompleted) before shutting down so
   // PSS clients don't think we're set up while we're shutting down.
   ClearPreferences();
diff --git a/chrome/browser/sync/profile_sync_service.h b/chrome/browser/sync/profile_sync_service.h
index 2b25e0c..63c04b8 100644
--- a/chrome/browser/sync/profile_sync_service.h
+++ b/chrome/browser/sync/profile_sync_service.h
@@ -325,6 +325,8 @@ class ProfileSyncService : public browser_sync::SyncFrontend,
   // any model-modifying operations.
   bool ShouldPushChanges();
 
+  const GURL& sync_service_url() const { return sync_service_url_; }
+
  protected:
   // Used by ProfileSyncServiceMock only.
   //
diff --git a/chrome/browser/sync/resources/about_sync.html b/chrome/browser/sync/resources/about_sync.html
index c65a7bb..6f62f5d 100644
--- a/chrome/browser/sync/resources/about_sync.html
+++ b/chrome/browser/sync/resources/about_sync.html
@@ -157,6 +157,12 @@ table.list#details .name {
           </div>
         </td>
       </tr>
+      <tr>
+        <td colspan="2">
+          <span class="desc"><h2> Sync URL </h2></span>
+          <strong jscontent="service_url"></strong>
+        </td>
+      </tr>
     </table>
     <br /><br /><br />
     <h2>  </h2>
-- 
cgit v1.1