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

Unified Diff: extensions/browser/content_verifier.cc

Issue 2824333003: Bind ContentVerifer::VerifyFailed as OnceCallback. (Closed)
Patch Set: Only ContentVerifyJob::FailureCallback Created 3 years, 8 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 | « extensions/browser/content_verifier.h ('k') | extensions/browser/content_verify_job.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/content_verifier.cc
diff --git a/extensions/browser/content_verifier.cc b/extensions/browser/content_verifier.cc
index 9fd012314b1b540f3188ba4584f07d3d42488539..a2e21d13919fc3b0e3d802510d6d1e91dc634dfe 100644
--- a/extensions/browser/content_verifier.cc
+++ b/extensions/browser/content_verifier.cc
@@ -125,7 +125,7 @@ ContentVerifyJob* ContentVerifier::CreateJobFor(
return new ContentVerifyJob(
new ContentHashReader(extension_id, data->version, extension_root,
normalized_path, delegate_->GetPublicKey()),
- base::Bind(&ContentVerifier::VerifyFailed, this, extension_id));
+ base::BindOnce(&ContentVerifier::VerifyFailed, this, extension_id));
}
void ContentVerifier::VerifyFailed(const std::string& extension_id,
@@ -207,9 +207,10 @@ void ContentVerifier::OnExtensionUnloaded(
fetcher_->ExtensionUnloaded(extension);
}
-void ContentVerifier::OnFetchCompleteHelper(const std::string& extension_id,
- bool shouldVerifyAnyPathsResult) {
- if (shouldVerifyAnyPathsResult)
+void ContentVerifier::OnFetchCompleteHelper(
+ const std::string& extension_id,
+ bool should_verify_any_paths_result) {
+ if (should_verify_any_paths_result)
delegate_->VerifyFailed(extension_id, ContentVerifyJob::MISSING_ALL_HASHES);
}
« no previous file with comments | « extensions/browser/content_verifier.h ('k') | extensions/browser/content_verify_job.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698