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

Unified Diff: chrome/browser/ui/app_list/drive/drive_app_converter.cc

Issue 308003005: app_list: Drive app integration. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 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/ui/app_list/drive/drive_app_converter.cc
diff --git a/chrome/browser/ui/app_list/drive/drive_app_converter.cc b/chrome/browser/ui/app_list/drive/drive_app_converter.cc
deleted file mode 100644
index 096a6f37cebfd151157eb6936374b86810ff86b8..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/app_list/drive/drive_app_converter.cc
+++ /dev/null
@@ -1,193 +0,0 @@
-// Copyright 2014 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/ui/app_list/drive/drive_app_converter.h"
-
-#include <algorithm>
-#include <set>
-
-#include "base/logging.h"
-#include "base/memory/ref_counted.h"
-#include "base/strings/utf_string_conversions.h"
-#include "chrome/browser/extensions/crx_installer.h"
-#include "chrome/browser/extensions/install_tracker.h"
-#include "chrome/browser/image_decoder.h"
-#include "chrome/browser/profiles/profile.h"
-#include "content/public/browser/browser_thread.h"
-#include "extensions/browser/extension_system.h"
-#include "extensions/common/constants.h"
-#include "net/base/load_flags.h"
-#include "net/http/http_status_code.h"
-#include "net/url_request/url_fetcher.h"
-#include "net/url_request/url_fetcher_delegate.h"
-#include "net/url_request/url_request_status.h"
-#include "third_party/skia/include/core/SkBitmap.h"
-
-using content::BrowserThread;
-
-// IconFetcher downloads |icon_url| using |converter|'s profile. The icon
-// url is passed from a DriveAppInfo and should follow icon url definition
-// in Drive API:
-// https://developers.google.com/drive/v2/reference/apps#resource
-// Each icon url represents a single image associated with a certain size.
-class DriveAppConverter::IconFetcher : public net::URLFetcherDelegate,
- public ImageDecoder::Delegate {
- public:
- IconFetcher(DriveAppConverter* converter,
- const GURL& icon_url,
- int expected_size)
- : converter_(converter),
- icon_url_(icon_url),
- expected_size_(expected_size) {}
- virtual ~IconFetcher() {
- if (image_decoder_.get())
- image_decoder_->set_delegate(NULL);
- }
-
- void Start() {
- fetcher_.reset(
- net::URLFetcher::Create(icon_url_, net::URLFetcher::GET, this));
- fetcher_->SetRequestContext(converter_->profile_->GetRequestContext());
- fetcher_->SetLoadFlags(net::LOAD_DO_NOT_SAVE_COOKIES);
- fetcher_->Start();
- }
-
- const GURL& icon_url() const { return icon_url_; }
- const SkBitmap& icon() const { return icon_; }
-
- private:
- // net::URLFetcherDelegate overrides:
- virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE {
- CHECK_EQ(fetcher_.get(), source);
- scoped_ptr<net::URLFetcher> fetcher(fetcher_.Pass());
-
- if (!fetcher->GetStatus().is_success() ||
- fetcher->GetResponseCode() != net::HTTP_OK) {
- converter_->OnIconFetchComplete(this);
- return;
- }
-
- std::string unsafe_icon_data;
- fetcher->GetResponseAsString(&unsafe_icon_data);
-
- image_decoder_ =
- new ImageDecoder(this, unsafe_icon_data, ImageDecoder::DEFAULT_CODEC);
- image_decoder_->Start(
- BrowserThread::GetMessageLoopProxyForThread(BrowserThread::UI));
- }
-
- // ImageDecoder::Delegate overrides:
- virtual void OnImageDecoded(const ImageDecoder* decoder,
- const SkBitmap& decoded_image) OVERRIDE {
- if (decoded_image.width() == expected_size_)
- icon_ = decoded_image;
- converter_->OnIconFetchComplete(this);
- }
-
- virtual void OnDecodeImageFailed(const ImageDecoder* decoder) OVERRIDE {
- converter_->OnIconFetchComplete(this);
- }
-
- DriveAppConverter* converter_;
- const GURL icon_url_;
- const int expected_size_;
-
- scoped_ptr<net::URLFetcher> fetcher_;
- scoped_refptr<ImageDecoder> image_decoder_;
- SkBitmap icon_;
-
- DISALLOW_COPY_AND_ASSIGN(IconFetcher);
-};
-
-DriveAppConverter::DriveAppConverter(Profile* profile,
- const drive::DriveAppInfo& app_info,
- const FinishedCallback& finished_callback)
- : profile_(profile),
- app_info_(app_info),
- app_(NULL),
- finished_callback_(finished_callback) {
- DCHECK(profile_);
-}
-
-DriveAppConverter::~DriveAppConverter() {
- PostInstallCleanUp();
-}
-
-void DriveAppConverter::Start() {
- if (app_info_.app_name.empty() ||
- !app_info_.create_url.is_valid()) {
- finished_callback_.Run(this, false);
- return;
- }
-
- web_app_.title = base::UTF8ToUTF16(app_info_.app_name);
- web_app_.app_url = app_info_.create_url;
-
- const std::set<int> allowed_sizes(extension_misc::kExtensionIconSizes,
- extension_misc::kExtensionIconSizes +
- extension_misc::kNumExtensionIconSizes);
- std::set<int> pending_sizes;
- for (size_t i = 0; i < app_info_.app_icons.size(); ++i) {
- const int icon_size = app_info_.app_icons[i].first;
- if (allowed_sizes.find(icon_size) == allowed_sizes.end() ||
- pending_sizes.find(icon_size) != pending_sizes.end()) {
- continue;
- }
-
- pending_sizes.insert(icon_size);
- const GURL& icon_url = app_info_.app_icons[i].second;
- IconFetcher* fetcher = new IconFetcher(this, icon_url, icon_size);
- fetchers_.push_back(fetcher); // Pass ownership to |fetchers|.
- fetcher->Start();
- }
-
- if (fetchers_.empty())
- StartInstall();
-}
-
-void DriveAppConverter::OnIconFetchComplete(const IconFetcher* fetcher) {
- const SkBitmap& icon = fetcher->icon();
- if (!icon.empty() && icon.width() != 0) {
- WebApplicationInfo::IconInfo icon_info;
- icon_info.url = fetcher->icon_url();
- icon_info.data = icon;
- icon_info.width = icon.width();
- icon_info.height = icon.height();
- web_app_.icons.push_back(icon_info);
- }
-
- fetchers_.erase(std::find(fetchers_.begin(), fetchers_.end(), fetcher));
-
- if (fetchers_.empty())
- StartInstall();
-}
-
-void DriveAppConverter::StartInstall() {
- DCHECK(!crx_installer_);
- crx_installer_ = extensions::CrxInstaller::CreateSilent(
- extensions::ExtensionSystem::Get(profile_)->extension_service());
- extensions::InstallTracker::Get(profile_)->AddObserver(this);
- crx_installer_->InstallWebApp(web_app_);
-}
-
-void DriveAppConverter::PostInstallCleanUp() {
- if (!crx_installer_)
- return;
-
- extensions::InstallTracker::Get(profile_)->RemoveObserver(this);
- crx_installer_ = NULL;
-}
-
-void DriveAppConverter::OnFinishCrxInstall(const std::string& extension_id,
- bool success) {
- if (!crx_installer_->extension() ||
- crx_installer_->extension()->id() != extension_id) {
- return;
- }
-
- app_ = crx_installer_->extension();
- finished_callback_.Run(this, success);
-
- PostInstallCleanUp();
-}

Powered by Google App Engine
This is Rietveld 408576698