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

Unified Diff: chrome/browser/bitmap_fetcher/bitmap_fetcher.cc

Issue 1097383005: Add possibility to define data callback to BitmapFetcher (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@r479598_extensions_content_verifier_directories_fail
Patch Set: Clarified how calling Init/Start twice works + fixed android Created 5 years, 6 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/bitmap_fetcher/bitmap_fetcher.cc
diff --git a/chrome/browser/bitmap_fetcher/bitmap_fetcher.cc b/chrome/browser/bitmap_fetcher/bitmap_fetcher.cc
index f99f3c20f56d889b94fee0598e5b2d58e9cf48f5..f379cff5f4f30ce0d26fbdceff32ca84f496bdb2 100644
--- a/chrome/browser/bitmap_fetcher/bitmap_fetcher.cc
+++ b/chrome/browser/bitmap_fetcher/bitmap_fetcher.cc
@@ -19,10 +19,10 @@ BitmapFetcher::BitmapFetcher(const GURL& url, BitmapFetcherDelegate* delegate)
BitmapFetcher::~BitmapFetcher() {
}
-void BitmapFetcher::Start(net::URLRequestContextGetter* request_context,
- const std::string& referrer,
- net::URLRequest::ReferrerPolicy referrer_policy,
- int load_flags) {
+void BitmapFetcher::Init(net::URLRequestContextGetter* request_context,
+ const std::string& referrer,
+ net::URLRequest::ReferrerPolicy referrer_policy,
+ int load_flags) {
if (url_fetcher_ != NULL)
return;
@@ -31,7 +31,11 @@ void BitmapFetcher::Start(net::URLRequestContextGetter* request_context,
url_fetcher_->SetReferrer(referrer);
url_fetcher_->SetReferrerPolicy(referrer_policy);
url_fetcher_->SetLoadFlags(load_flags);
- url_fetcher_->Start();
+}
+
+void BitmapFetcher::Start() {
+ if (url_fetcher_)
+ url_fetcher_->Start();
}
// Methods inherited from URLFetcherDelegate.
« no previous file with comments | « chrome/browser/bitmap_fetcher/bitmap_fetcher.h ('k') | chrome/browser/bitmap_fetcher/bitmap_fetcher_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698