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

Unified Diff: extensions/browser/content_verify_job.cc

Issue 2572833004: Only whitelist messages.json files in _locales for content verification (Closed)
Patch Set: turn on logging to try and understand win_chromium_x64_rel_ng test failures Created 4 years 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 | « extensions/browser/content_verifier.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/content_verify_job.cc
diff --git a/extensions/browser/content_verify_job.cc b/extensions/browser/content_verify_job.cc
index 4eee6a072da0182bd78417bfb0545d32bd611fee..b7f10b836a6b551f5ea672d99e75f551bdfaad56 100644
--- a/extensions/browser/content_verify_job.cc
+++ b/extensions/browser/content_verify_job.cc
@@ -208,9 +208,9 @@ void ContentVerifyJob::DispatchFailureCallback(FailureReason reason) {
DCHECK(!failed_);
failed_ = true;
if (!failure_callback_.is_null()) {
- VLOG(1) << "job failed for " << hash_reader_->extension_id() << " "
- << hash_reader_->relative_path().MaybeAsASCII()
- << " reason:" << reason;
+ LOG(WARNING) << "job failed for " << hash_reader_->extension_id() << " "
+ << hash_reader_->relative_path().MaybeAsASCII()
+ << " reason:" << reason;
failure_callback_.Run(reason);
failure_callback_.Reset();
}
« no previous file with comments | « extensions/browser/content_verifier.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698