Index: chrome/browser/apps/drive/drive_app_converter.cc |
diff --git a/chrome/browser/ui/app_list/drive/drive_app_converter.cc b/chrome/browser/apps/drive/drive_app_converter.cc |
similarity index 90% |
rename from chrome/browser/ui/app_list/drive/drive_app_converter.cc |
rename to chrome/browser/apps/drive/drive_app_converter.cc |
index 096a6f37cebfd151157eb6936374b86810ff86b8..1d4362aeac3c2854ad678066637e9613b152f00e 100644 |
--- a/chrome/browser/ui/app_list/drive/drive_app_converter.cc |
+++ b/chrome/browser/apps/drive/drive_app_converter.cc |
@@ -2,7 +2,7 @@ |
// 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 "chrome/browser/apps/drive/drive_app_converter.h" |
#include <algorithm> |
#include <set> |
@@ -115,8 +115,9 @@ DriveAppConverter::~DriveAppConverter() { |
} |
void DriveAppConverter::Start() { |
- if (app_info_.app_name.empty() || |
- !app_info_.create_url.is_valid()) { |
+ DCHECK(!IsStarted()); |
+ |
+ if (app_info_.app_name.empty() || !app_info_.create_url.is_valid()) { |
finished_callback_.Run(this, false); |
return; |
} |
@@ -146,6 +147,10 @@ void DriveAppConverter::Start() { |
StartInstall(); |
} |
+bool DriveAppConverter::IsStarted() const { |
+ return !fetchers_.empty() || crx_installer_; |
+} |
+ |
void DriveAppConverter::OnIconFetchComplete(const IconFetcher* fetcher) { |
const SkBitmap& icon = fetcher->icon(); |
if (!icon.empty() && icon.width() != 0) { |
@@ -167,6 +172,13 @@ void DriveAppConverter::StartInstall() { |
DCHECK(!crx_installer_); |
crx_installer_ = extensions::CrxInstaller::CreateSilent( |
extensions::ExtensionSystem::Get(profile_)->extension_service()); |
+ crx_installer_->set_creation_flags(extensions::Extension::FROM_BOOKMARK); |
+ // The converted url app should not be syncable. Drive apps go with the user's |
+ // account and url apps will be created when needed. Syncing those apps could |
+ // hit an edge case where the synced url apps become orphans when the user has |
+ // corresponding chrome apps. |
+ crx_installer_->set_is_syncable(false); |
+ |
extensions::InstallTracker::Get(profile_)->AddObserver(this); |
crx_installer_->InstallWebApp(web_app_); |
} |
@@ -187,7 +199,8 @@ void DriveAppConverter::OnFinishCrxInstall(const std::string& extension_id, |
} |
app_ = crx_installer_->extension(); |
- finished_callback_.Run(this, success); |
- |
PostInstallCleanUp(); |
+ |
+ finished_callback_.Run(this, success); |
+ // |finished_callback_| could delete this. |
} |