Add/resurrect support for bundles of WebStore items.

chrome/browser/safe_image_fetcher.cc

+// Copyright 2015 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/safe_image_fetcher.h"
+
+#include "base/bind.h"
+#include "base/callback.h"
+#include "base/message_loop/message_loop_proxy.h"
+#include "net/base/load_flags.h"
+#include "net/url_request/url_fetcher.h"
+#include "third_party/skia/include/core/SkBitmap.h"
+
+SafeImageFetcher::SafeImageFetcher(
+    const GURL& url,
+    net::URLRequestContextGetter* context_getter,
+    const GetImageCallback& callback)
+    : ImageRequest(base::MessageLoopProxy::current()),
+      callback_(callback),
+      weak_ptr_factory_(this) {
+  if (url.is_valid()) {
+    DCHECK(context_getter);
+    url_fetcher_.reset(net::URLFetcher::Create(
+        url, net::URLFetcher::GET, this));
+    url_fetcher_->SetRequestContext(context_getter);
+    url_fetcher_->SetLoadFlags(net::LOAD_DO_NOT_SAVE_COOKIES |
+                               net::LOAD_DO_NOT_SEND_COOKIES);
+
+    url_fetcher_->Start();
+    // We'll get called back in OnURLFetchComplete.
+  } else {
+    base::MessageLoopProxy::current()->PostTask(
+        FROM_HERE,
+        base::Bind(&SafeImageFetcher::OnDecodeImageFailed,
+                   weak_ptr_factory_.GetWeakPtr()));
+  }
+}
+
+SafeImageFetcher::~SafeImageFetcher() {
+}
+
+void SafeImageFetcher::OnURLFetchComplete(const net::URLFetcher* source) {
+  DCHECK_EQ(url_fetcher_.get(), source);
+  int response_code =
+      source->GetStatus().is_success() ? source->GetResponseCode() : 0;
+  if (!source->GetStatus().is_success() ||
+      response_code / 100 == 4 || response_code / 100 == 5) {

[Lei Zhang, 2015/03/31 09:39:08]

So if we somehow get respond code 900, that counts as a success?

[Marc Treib, 2015/03/31 12:48:21]

Hm, I'm guessing is_success() wouldn't be true in this case? I just copied this
code from webstore_install_helper.cc. Anyway, obsolete now that the class is
removed again. :)

+    OnDecodeImageFailed();
+  } else {
+    std::string response_data;
+    source->GetResponseAsString(&response_data);
+
+    ImageDecoder::Start(this, response_data);
+  }
+  url_fetcher_.reset();
+}
+
+void SafeImageFetcher::OnImageDecoded(const SkBitmap& decoded_image) {
+  callback_.Run(decoded_image);
+}
+
+void SafeImageFetcher::OnDecodeImageFailed() {
+  callback_.Run(SkBitmap());
+}