diff options
author | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-15 08:24:59 +0000 |
---|---|---|
committer | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-15 08:24:59 +0000 |
commit | f36ed64560f4b52724d223bc870addc5a5e3b73c (patch) | |
tree | a57f393275bf6b24bb8b3d31d93a587544cd0cb9 /chrome/browser/extensions/crx_installer.cc | |
parent | 7e5d1e689c3c5a8858a9675df46ca327f51b7c6e (diff) | |
download | chromium_src-f36ed64560f4b52724d223bc870addc5a5e3b73c.zip chromium_src-f36ed64560f4b52724d223bc870addc5a5e3b73c.tar.gz chromium_src-f36ed64560f4b52724d223bc870addc5a5e3b73c.tar.bz2 |
Revert 36337 and 36338. Compile failure.
TBR=erikkay@chromium.org
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36339 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/crx_installer.cc')
-rw-r--r-- | chrome/browser/extensions/crx_installer.cc | 25 |
1 files changed, 1 insertions, 24 deletions
diff --git a/chrome/browser/extensions/crx_installer.cc b/chrome/browser/extensions/crx_installer.cc index defeff4..ca01297 100644 --- a/chrome/browser/extensions/crx_installer.cc +++ b/chrome/browser/extensions/crx_installer.cc @@ -5,7 +5,6 @@ #include "chrome/browser/extensions/crx_installer.h" #include "app/l10n_util.h" -#include "app/resource_bundle.h" #include "base/file_util.h" #include "base/scoped_temp_dir.h" #include "base/string_util.h" @@ -14,12 +13,9 @@ #include "chrome/browser/chrome_thread.h" #include "chrome/browser/extensions/convert_user_script.h" #include "chrome/browser/extensions/extension_file_util.h" -#include "chrome/browser/shell_integration.h" -#include "chrome/browser/web_applications/web_app.h" #include "chrome/common/extensions/extension_error_reporter.h" #include "chrome/common/notification_service.h" #include "chrome/common/notification_type.h" -#include "grit/browser_resources.h" #include "grit/chromium_strings.h" #include "third_party/skia/include/core/SkBitmap.h" @@ -157,11 +153,10 @@ void CrxInstaller::OnUnpackSuccess(const FilePath& temp_dir, return; } - if (client_.get() || extension_->IsApp()) { + if (client_.get()) { Extension::DecodeIcon(extension_.get(), Extension::EXTENSION_ICON_LARGE, &install_icon_); } - ChromeThread::PostTask( ChromeThread::UI, FROM_HERE, NewRunnableMethod(this, &CrxInstaller::ConfirmInstall)); @@ -237,24 +232,6 @@ void CrxInstaller::CompleteInstall() { return; } - if (extension_->IsApp()) { - SkBitmap icon = install_icon_.get() ? *install_icon_ : - *ResourceBundle::GetSharedInstance().GetBitmapNamed( - IDR_EXTENSION_DEFAULT_ICON); - - ShellIntegration::ShortcutInfo shortcut_info; - shortcut_info.url = extension_->app_launch_url(); - shortcut_info.title = UTF8ToUTF16(extension_->name()); - shortcut_info.description = UTF8ToUTF16(extension_->description()); - shortcut_info.favicon = icon; - shortcut_info.create_on_desktop = true; - - // TODO(aa): Seems nasty to be reusing the old webapps code this way. What - // baggage am I inheriting? - web_app::CreateShortcut(frontend_->profile()->GetPath(), shortcut_info, - NULL); - } - // This is lame, but we must reload the extension because absolute paths // inside the content scripts are established inside InitFromValue() and we // just moved the extension. |