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

Unified Diff: chrome/browser/notifications/sync_notifier/notification_bitmap_fetcher.cc

Issue 15295018: Continue bitmap fetching for notifications. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Synced Notification Bitmap Fetching - Linux Compile 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/notifications/sync_notifier/notification_bitmap_fetcher.cc
diff --git a/chrome/browser/notifications/sync_notifier/notification_bitmap_fetcher.cc b/chrome/browser/notifications/sync_notifier/notification_bitmap_fetcher.cc
new file mode 100644
index 0000000000000000000000000000000000000000..05c4d12e2901c3e47ec5888f9239930e4976b88d
--- /dev/null
+++ b/chrome/browser/notifications/sync_notifier/notification_bitmap_fetcher.cc
@@ -0,0 +1,86 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/notifications/sync_notifier/notification_bitmap_fetcher.h"
+
+#include "content/public/browser/browser_thread.h"
+#include "net/url_request/url_fetcher.h"
+
+namespace notifier {
+
+NotificationBitmapFetcher::NotificationBitmapFetcher(
+ const GURL& url,
+ NotificationBitmapFetcherDelegate* delegate)
+ : url_(url), delegate_(delegate) {}
+
+NotificationBitmapFetcher::~NotificationBitmapFetcher() {}
+
+void NotificationBitmapFetcher::Start() {
+ if (url_fetcher_ == NULL) {
+ url_fetcher_.reset(
+ net::URLFetcher::Create(url_, net::URLFetcher::GET, this));
+ }
+ url_fetcher_->Start();
+}
+
+void NotificationBitmapFetcher::SetURLFetcherForTest(
+ scoped_ptr<net::URLFetcher> url_fetcher) {
+ url_fetcher_ = url_fetcher.Pass();
+}
+
+// Methods inherited from URLFetcherDelegate.
+
+void NotificationBitmapFetcher::OnURLFetchComplete(
+ const net::URLFetcher* source) {
+ std::string image_data;
+
+ DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
+
+ // Copy the data into the string. Keep in mind it may have embedded nulls.
+ source->GetResponseAsString(&image_data);
+
+ // Handle fetch failure. If it failed, set failed to true, and fire
+ // notification to listeners.
+ if (image_data.length() == 0) {
+ OnDecodeImageFailed(NULL);
+ return;
+ }
+
+ // Create an ImageDecoder with the data and assign it to the refptr.
+ image_decoder_ = new ImageDecoder(this, image_data,
+ ImageDecoder::DEFAULT_CODEC);
+
+ // Call start to begin decoding. The ImageDecoder will call OnImageDecoded
+ // with the data when it is done.
+ scoped_refptr<base::MessageLoopProxy> task_runner =
+ content::BrowserThread::GetMessageLoopProxyForThread(
+ content::BrowserThread::UI);
+ image_decoder_->Start(task_runner);
+}
+
+void NotificationBitmapFetcher::OnURLFetchDownloadProgress(
+ const net::URLFetcher* source, int64 current, int64 total) {
+ // Do nothing here.
+}
+
+// Methods inherited from ImageDecoder::Delegate.
+
+void NotificationBitmapFetcher::OnImageDecoded(
+ const ImageDecoder* decoder, const SkBitmap& decoded_image) {
+ // Make a copy of the bitmap which we pass back to the UI thread.
+ bitmap_.reset(new SkBitmap());
+ decoded_image.deepCopyTo(bitmap_.get(), decoded_image.getConfig());
+
+ // Report success.
+ delegate_->OnFetchComplete(bitmap_.get());
+}
+
+void NotificationBitmapFetcher::OnDecodeImageFailed(
+ const ImageDecoder* decoder) {
+
+ // Report failure.
+ delegate_->OnFetchComplete(NULL);
+}
+
+} // namespace notifier

Powered by Google App Engine
This is Rietveld 408576698