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

Unified Diff: chrome/browser/ui/app_list/search/webstore/webstore_installer.cc

Issue 298303002: Add option to install an ephemeral app to ChromeOS shelf context menu (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added browser test Created 6 years, 7 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/search/webstore/webstore_installer.cc
diff --git a/chrome/browser/ui/app_list/search/webstore/webstore_installer.cc b/chrome/browser/ui/app_list/search/webstore/webstore_installer.cc
index 0b4fc6ee3ff360833fd2ac6b681b5a173945ac6a..d378eb34b7d6460352421d05f3878040751e216f 100644
--- a/chrome/browser/ui/app_list/search/webstore/webstore_installer.cc
+++ b/chrome/browser/ui/app_list/search/webstore/webstore_installer.cc
@@ -4,35 +4,20 @@
#include "chrome/browser/ui/app_list/search/webstore/webstore_installer.h"
-#include "chrome/browser/extensions/extension_install_prompt.h"
-#include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/scoped_tabbed_browser_displayer.h"
-
namespace app_list {
WebstoreInstaller::WebstoreInstaller(const std::string& webstore_item_id,
Profile* profile,
gfx::NativeWindow parent_window,
const Callback& callback)
- : WebstoreStartupInstaller(webstore_item_id, profile, true, callback),
- profile_(profile),
- parent_window_(parent_window) {
+ : WebstoreInstallPrompt(webstore_item_id,
+ profile,
+ parent_window,
+ callback) {
set_install_source(
extensions::WebstoreInstaller::INSTALL_SOURCE_APP_LAUNCHER);
}
WebstoreInstaller::~WebstoreInstaller() {}
-scoped_ptr<ExtensionInstallPrompt> WebstoreInstaller::CreateInstallUI() {
- return make_scoped_ptr(
- new ExtensionInstallPrompt(profile_, parent_window_, this));
-}
-
-content::WebContents* WebstoreInstaller::OpenURL(
- const content::OpenURLParams& params) {
- chrome::ScopedTabbedBrowserDisplayer displayer(
- profile_, chrome::GetActiveDesktop());
- return displayer.browser()->OpenURL(params);
-}
-
} // namespace app_list

Powered by Google App Engine
This is Rietveld 408576698