Eliminate all UI thread decoding of extension images....

chrome/browser/extensions/extension_action_context_menu_model.cc

-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 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/extensions/extension_action_context_menu_model.h"
 
 #include "app/l10n_util.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/extensions/extension_tabs_module.h"
 #include "chrome/browser/extensions/extensions_service.h"
@@ skipping 91 matching lines @@
       DCHECK(!extension_->options_url().is_empty());
       profile->GetExtensionProcessManager()->OpenOptionsPage(extension_,
                                                              browser);
       break;
     case DISABLE: {
       ExtensionsService* extension_service = profile->GetExtensionsService();
       extension_service->DisableExtension(extension_->id());
       break;
     }
     case UNINSTALL: {
-      scoped_ptr<SkBitmap> uninstall_icon;
-      Extension::DecodeIcon(extension_, Extension::EXTENSION_ICON_LARGE,
-                            &uninstall_icon);
-
-      ExtensionInstallUI client(profile);
-      client.ConfirmUninstall(this, extension_, uninstall_icon.get());
+      install_ui_.reset(new ExtensionInstallUI(profile));
+      install_ui_->ConfirmUninstall(this, extension_);
       break;
     }
     case MANAGE: {
       browser->OpenURL(GURL(chrome::kChromeUIExtensionsURL), GURL(),
                        SINGLETON_TAB, PageTransition::LINK);
       break;
     }
     case INSPECT_POPUP: {
       if (delegate_)
         delegate_->ShowPopupForDevToolsWindow(extension_, extension_action_);
       break;
     }
     default:
      NOTREACHED() << "Unknown option";
      break;
   }
 }
 
 void ExtensionActionContextMenuModel::InstallUIProceed(bool create_app) {
   DCHECK(!create_app);
 
   // TODO(finnur): GetLastActive returns NULL in unit tests.
   Browser* browser = BrowserList::GetLastActive();
   std::string id = extension_->id();
   browser->profile()->GetExtensionsService()->UninstallExtension(id, false);
 }