Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(170)

Unified Diff: chrome/browser/sync_file_system/drive_file_sync_service.cc

Issue 15657002: Mirror syncfs log to console and WebUI, with LogSeverity support. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: expect -> assert fix Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync_file_system/drive_file_sync_service.cc
diff --git a/chrome/browser/sync_file_system/drive_file_sync_service.cc b/chrome/browser/sync_file_system/drive_file_sync_service.cc
index 07944d441401e9ec7dae6d2cbaa3391d00e995e5..2f6031b186694432856d4c7a6fafe6ad3637cd7c 100644
--- a/chrome/browser/sync_file_system/drive_file_sync_service.cc
+++ b/chrome/browser/sync_file_system/drive_file_sync_service.cc
@@ -1476,7 +1476,9 @@ void DriveFileSyncService::MaybeStartFetchChanges() {
}
void DriveFileSyncService::OnNotificationReceived() {
- util::Log("Notification received to check for Google Drive updates");
+ util::Log(logging::LOG_INFO,
+ FROM_HERE,
+ "Notification received to check for Google Drive updates");
// TODO(calvinlo): Try to eliminate may_have_unfetched_changes_ variable.
may_have_unfetched_changes_ = true;
MaybeStartFetchChanges();
@@ -1484,7 +1486,9 @@ void DriveFileSyncService::OnNotificationReceived() {
void DriveFileSyncService::OnPushNotificationEnabled(bool enabled) {
const char* status = (enabled ? "enabled" : "disabled");
- util::Log("XMPP Push notification is %s", status);
+ util::Log(logging::LOG_INFO,
+ FROM_HERE,
+ "XMPP Push notification is %s", status);
}
void DriveFileSyncService::MaybeScheduleNextTask() {
@@ -1606,12 +1610,17 @@ void DriveFileSyncService::DidFetchChangesForIncrementalSync(
}
if (reset_sync_root) {
- LOG(WARNING) << "Detected unexpected SyncRoot deletion.";
+ util::Log(logging::LOG_WARNING,
+ FROM_HERE,
+ "Detected unexpected SyncRoot deletion.");
metadata_store_->SetSyncRootDirectory(std::string());
}
for (std::set<GURL>::iterator itr = reset_origins.begin();
itr != reset_origins.end(); ++itr) {
- LOG(WARNING) << "Detected unexpected OriginRoot deletion:" << itr->spec();
+ util::Log(logging::LOG_WARNING,
+ FROM_HERE,
+ "Detected unexpected OriginRoot deletion: %s",
+ itr->spec().c_str());
pending_batch_sync_origins_.erase(*itr);
metadata_store_->SetOriginRootDirectory(*itr, std::string());
}
« no previous file with comments | « chrome/browser/sync_file_system/drive/api_util.cc ('k') | chrome/browser/sync_file_system/drive_file_sync_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698