Add option to install an ephemeral app to ChromeOS shelf context menu

chrome/browser/ui/ash/launcher/app_window_launcher_item_controller.cc

 // 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/ash/launcher/app_window_launcher_item_controller.h"
 
 #include "apps/app_window.h"
 #include "apps/ui/native_app_window.h"
 #include "ash/shelf/shelf_model.h"
 #include "ash/wm/window_state.h"
 #include "ash/wm/window_util.h"
+#include "chrome/browser/extensions/webstore_install_prompt.h"
 #include "chrome/browser/ui/ash/launcher/chrome_launcher_app_menu_item.h"
 #include "chrome/browser/ui/ash/launcher/chrome_launcher_app_menu_item_v2app.h"
 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller.h"
 #include "chrome/browser/ui/ash/launcher/launcher_application_menu_item_model.h"
 #include "chrome/browser/ui/ash/launcher/launcher_context_menu.h"
 #include "chrome/browser/ui/ash/launcher/launcher_item_controller.h"
 #include "content/public/browser/web_contents.h"
 #include "skia/ext/image_operations.h"
 #include "ui/aura/client/aura_constants.h"
 #include "ui/aura/window.h"
@@ skipping 122 matching lines @@
 }
 
 void AppWindowLauncherItemController::ActivateIndexedApp(size_t index) {
   if (index >= app_windows_.size())
     return;
   AppWindowList::iterator it = app_windows_.begin();
   std::advance(it, index);
   ShowAndActivateOrMinimize(*it);
 }
 
+void AppWindowLauncherItemController::InstallApp() {
+  AppWindow* parent_window =
+      last_active_app_window_ ? last_active_app_window_ : app_windows_.back();
+  scoped_refptr<extensions::WebstoreInstallPrompt> installer =
+      new extensions::WebstoreInstallPrompt(
+          app_id(),
+          launcher_controller()->profile(),
+          parent_window ? parent_window->GetNativeWindow() : NULL,

[Mr4D (OOO till 08-26), 2014/05/28 15:42:15]

I am not quite sure that I understand why you need this window. It feels like
that you are using a "random" window as a parent (which might even be hidden).
You might therefore be better off to use e.g. the shelf's window instead?

[tmdiep, 2014/05/28 23:22:20]

The purpose of the window is for positioning the install dialog. Ideally, I
would use the foreground window of the app. If there is no visible window, we
can just pass in NULL for the parent and it will center the dialog on the
screen.

Good point about hidden windows; I'll refactor the parent window selection.

+          extensions::WebstoreInstallPrompt::Callback());
+  installer->BeginInstall();
+}
+
 ChromeLauncherAppMenuItems AppWindowLauncherItemController::GetApplicationList(
     int event_flags) {
   ChromeLauncherAppMenuItems items;
   items.push_back(new ChromeLauncherAppMenuItem(GetTitle(), NULL, false));
   int index = 0;
   for (AppWindowList::iterator iter = app_windows_.begin();
        iter != app_windows_.end();
        ++iter) {
     AppWindow* app_window = *iter;
     scoped_ptr<gfx::Image> image(GetAppListIcon(app_window));
@@ skipping 115 matching lines @@
   }
   if (window_to_show->GetBaseWindow()->IsActive()) {
     // Coming here, only a single window is active. For keyboard activations
     // the window gets animated.
     AnimateWindow(window_to_show->GetNativeWindow(),
                   wm::WINDOW_ANIMATION_TYPE_BOUNCE);
   } else {
     ShowAndActivateOrMinimize(window_to_show);
   }
 }