 Chromium Code Reviews
 Chromium Code Reviews Issue 1756193008:
  Support uninstalling ARC app from Chrome launcher  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 1756193008:
  Support uninstalling ARC app from Chrome launcher  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: chrome/browser/ui/app_list/arc/arc_app_context_menu.h | 
| diff --git a/chrome/browser/ui/app_list/arc/arc_app_context_menu.h b/chrome/browser/ui/app_list/arc/arc_app_context_menu.h | 
| index 7e7acbc5ee45d1539ef6e65fbf628c750a29c134..abf17ec9b068013f6af6e53b3f294092f77abecb 100644 | 
| --- a/chrome/browser/ui/app_list/arc/arc_app_context_menu.h | 
| +++ b/chrome/browser/ui/app_list/arc/arc_app_context_menu.h | 
| @@ -17,6 +17,10 @@ namespace app_list { | 
| class AppContextMenuDelegate; | 
| } | 
| +namespace mojo { | 
| +class String; | 
| +} | 
| + | 
| class ArcAppContextMenu : public app_list::AppContextMenu { | 
| public: | 
| ArcAppContextMenu(app_list::AppContextMenuDelegate* delegate, | 
| @@ -33,6 +37,11 @@ class ArcAppContextMenu : public app_list::AppContextMenu { | 
| void ExecuteCommand(int command_id, int event_flags) override; | 
| private: | 
| + void UninstallApp(); | 
| + static void OnUninstallAppResponse(const mojo::String& error_message); | 
| 
khmel
2016/03/07 21:11:52
I don't think ArcAppContextMenu is good place to h
 
victorhsieh0
2016/03/07 23:54:43
Moved to utils per discussion.
 | 
| + | 
| + bool CanBeUninstalled() const; | 
| + | 
| DISALLOW_COPY_AND_ASSIGN(ArcAppContextMenu); | 
| }; |