Mirror syncfs log to console and WebUI, with LogSeverity support.

chrome/browser/sync_file_system/sync_file_system_service.cc

 // 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.
 
 #include "chrome/browser/sync_file_system/sync_file_system_service.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/memory/ref_counted.h"
 #include "base/stl_util.h"
 #include "chrome/browser/extensions/api/sync_file_system/extension_sync_event_observer.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/sync/profile_sync_service_factory.h"
 #include "chrome/browser/sync_file_system/drive_file_sync_service.h"
 #include "chrome/browser/sync_file_system/local_file_sync_service.h"
+#include "chrome/browser/sync_file_system/logger.h"
 #include "chrome/browser/sync_file_system/sync_event_observer.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/extensions/extension.h"
 #include "components/browser_context_keyed_service/browser_context_dependency_manager.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_service.h"
 #include "googleurl/src/gurl.h"
 #include "webkit/fileapi/file_system_context.h"
 #include "webkit/fileapi/syncable/sync_direction.h"
@@ skipping 31 matching lines @@
     extension_misc::UnloadedExtensionReason reason,
     const GURL& origin,
     SyncStatusCode code) {
   DCHECK(chrome::NOTIFICATION_EXTENSION_UNLOADED == type);
   DCHECK(extension_misc::UNLOAD_REASON_DISABLE == reason ||
          extension_misc::UNLOAD_REASON_UNINSTALL == reason);
   if (code != SYNC_STATUS_OK &&
       code != SYNC_STATUS_UNKNOWN_ORIGIN) {
     switch (reason) {
       case extension_misc::UNLOAD_REASON_DISABLE:
-        LOG(WARNING) << "Disabling origin for UNLOAD(DISABLE) failed: "
-                     << origin.spec();
+        util::Log(logging::LOG_WARNING,
+                  "Disabling origin for UNLOAD(DISABLE) failed: %s",
+                  origin.spec().c_str());

[tzik, 2013/05/22 09:30:54]

This replacement drops the location of the log (e.g. file name, line number and
function name).
Could you add a macro to keep them?

[calvinlo, 2013/05/23 10:00:15]

Thanks for pointing this out and suggesting the implementation. I think I have
this working now.

         break;
       case extension_misc::UNLOAD_REASON_UNINSTALL:
-        LOG(WARNING) << "Uninstall origin for UNLOAD(UNINSTALL) failed: "
-                     << origin.spec();
+        util::Log(logging::LOG_WARNING,
+                  "Uninstall origin for UNLOAD(UNINSTALL) failed: %s",
+                  origin.spec().c_str());
         break;
       default:
         break;
     }
   }
 }
 
 void DidHandleOriginForExtensionEnabledEvent(
     int type,
     const GURL& origin,
     SyncStatusCode code) {
   DCHECK(chrome::NOTIFICATION_EXTENSION_ENABLED == type);
   if (code != SYNC_STATUS_OK)
-    LOG(WARNING) << "Enabling origin for ENABLED failed: " << origin.spec();
+    util::Log(logging::LOG_WARNING,
+              "Enabling origin for ENABLED failed: %s",
+              origin.spec().c_str());
 }
 
 }  // namespace
 
 void SyncFileSystemService::Shutdown() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
   local_file_service_->Shutdown();
   local_file_service_.reset();
 
@@ skipping 460 matching lines @@
       syncer::APPS);
   remote_file_service_->SetSyncEnabled(sync_enabled_);
   if (sync_enabled_) {
     base::MessageLoopProxy::current()->PostTask(
         FROM_HERE, base::Bind(&SyncFileSystemService::MaybeStartSync,
                               AsWeakPtr()));
   }
 }
 
 }  // namespace sync_file_system