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

Unified Diff: chrome/browser/chromeos/drive/change_list_loader.cc

Issue 20609006: Add severity info to drive::EventLogger. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase - the 2nd try - Created 7 years, 5 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
« no previous file with comments | « no previous file | chrome/browser/chromeos/drive/drive_integration_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/drive/change_list_loader.cc
diff --git a/chrome/browser/chromeos/drive/change_list_loader.cc b/chrome/browser/chromeos/drive/change_list_loader.cc
index 531e231cd7dc07c7780bb8f71651fd1fbcf26c0c..bd0f4236b22a4c0a486f5edad1d11d5f9c02090c 100644
--- a/chrome/browser/chromeos/drive/change_list_loader.cc
+++ b/chrome/browser/chromeos/drive/change_list_loader.cc
@@ -72,7 +72,7 @@ void ChangeListLoader::CheckForUpdates(const FileOperationCallback& callback) {
// We only start to check for updates iff the load is done.
// I.e., we ignore checking updates if not loaded to avoid starting the
// load without user's explicit interaction (such as opening Drive).
- util::Log("Checking for updates");
+ util::Log(logging::LOG_INFO, "Checking for updates");
Load(DirectoryFetchInfo(), callback);
}
}
@@ -212,7 +212,8 @@ void ChangeListLoader::OnDirectoryLoadComplete(
FileError error) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- util::Log("Fast-fetch complete: %s => %s",
+ util::Log(logging::LOG_INFO,
+ "Fast-fetch complete: %s => %s",
directory_fetch_info.ToString().c_str(),
FileErrorToString(error).c_str());
const std::string& resource_id = directory_fetch_info.resource_id();
@@ -440,7 +441,8 @@ void ChangeListLoader::CheckChangestampAndLoadDirectoryIfNeeded(
// enough, but we log this message here, so "Fast-fetch start" and
// "Fast-fetch complete" always match.
// TODO(satorux): Distinguish the "not fetching at all" case.
- util::Log("Fast-fetch start: %s; Server changestamp: %s",
+ util::Log(logging::LOG_INFO,
+ "Fast-fetch start: %s; Server changestamp: %s",
directory_fetch_info.ToString().c_str(),
base::Int64ToString(last_known_remote_changestamp_).c_str());
@@ -630,7 +632,9 @@ void ChangeListLoader::UpdateFromChangeList(
// the initial content retrieval.
const bool should_notify_changed_directories = is_delta_update;
- util::Log("Apply change lists (is delta: %d)", is_delta_update);
+ util::Log(logging::LOG_INFO,
+ "Apply change lists (is delta: %d)",
+ is_delta_update);
blocking_task_runner_->PostTaskAndReply(
FROM_HERE,
base::Bind(&ChangeListProcessor::Apply,
@@ -656,7 +660,8 @@ void ChangeListLoader::UpdateFromChangeListAfterApply(
DCHECK(!callback.is_null());
const base::TimeDelta elapsed = base::Time::Now() - start_time;
- util::Log("Change lists applied (elapsed time: %sms)",
+ util::Log(logging::LOG_INFO,
+ "Change lists applied (elapsed time: %sms)",
base::Int64ToString(elapsed.InMilliseconds()).c_str());
if (should_notify_changed_directories) {
« no previous file with comments | « no previous file | chrome/browser/chromeos/drive/drive_integration_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698