Add option to install an ephemeral app to the Windows jump list

chrome/browser/ui/views/apps/native_app_window_views_win.h

Index: chrome/browser/ui/views/apps/native_app_window_views_win.h
diff --git a/chrome/browser/ui/views/apps/native_app_window_views_win.h b/chrome/browser/ui/views/apps/native_app_window_views_win.h
index 17ec74967aff344167b0b81a2c687b197dae3b35..1f7e7c0df6db465b920686415db6dd60ddb5d9e5 100644
--- a/chrome/browser/ui/views/apps/native_app_window_views_win.h
+++ b/chrome/browser/ui/views/apps/native_app_window_views_win.h
@@ -5,8 +5,11 @@
 #ifndef CHROME_BROWSER_UI_VIEWS_APPS_NATIVE_APP_WINDOW_VIEWS_WIN_H_
 #define CHROME_BROWSER_UI_VIEWS_APPS_NATIVE_APP_WINDOW_VIEWS_WIN_H_
 
+#include "chrome/browser/shell_integration.h"
 #include "chrome/browser/ui/views/apps/native_app_window_views.h"
 
+class JumpListUpdater;
+
 // Windows-specific parts of the views-backed native shell window implementation
 // for packaged apps.
 class NativeAppWindowViewsWin : public NativeAppWindowViews {
@@ -19,11 +22,25 @@ class NativeAppWindowViewsWin : public NativeAppWindowViews {
   // Overridden from NativeAppWindowViews:
   virtual void OnBeforeWidgetInit(views::Widget::InitParams* init_params,
                                   views::Widget* widget) OVERRIDE;
+  virtual void InitializeDefaultWindow(
+      const apps::ShellWindow::CreateParams& create_params) OVERRIDE;
 
   // Overridden from ui::BaseWindow:
   virtual void Show() OVERRIDE;
   virtual void Activate() OVERRIDE;
 
+  // Overridden from apps::NativeAppWindow:
+  virtual void UpdateAppMenu() OVERRIDE;
+
+  void OnShortcutInfoLoaded(

[tapted, 2014/02/12 10:16:28]

nit: move non-overrides up below ActivateParentDesktopIfNecessary

+      const ShellIntegration::ShortcutInfo& shortcut_info);
+
+  HWND GetNativeAppWindowHWND() const;
+
+  base::WeakPtrFactory<NativeAppWindowViewsWin> weak_ptr_factory_;
+
+  scoped_ptr<JumpListUpdater> jumplist_updater_;

[tapted, 2014/02/12 10:16:28]

I don't think this is really needed - it doesn't hold any state that can't just
be fetched each time in UpdateAppMenu(). The only bit that really feels
heavyweight is generating the app_id/taskbar group, and this isn't specific to
JumpListUpdater, so it probably belongs here instead. (i.e. replace this data
member with `wstring taskbar_app_id` or similar)

I think if we introduce a ~"modify jumplist" API then we will need some extra
state, but it shouldn't be windows-specific (i.e. would go on a parent class,
and be more generic).

+
   DISALLOW_COPY_AND_ASSIGN(NativeAppWindowViewsWin);
 };