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

Unified Diff: chrome/browser/ui/ash/launcher/chrome_launcher_controller_mus.h

Issue 2718563008: mash: Use mojo for ShelfItemDelegate and [app] MenuItem. (Closed)
Patch Set: Address comments. Created 3 years, 9 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/ash/launcher/chrome_launcher_controller_mus.h
diff --git a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_mus.h b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_mus.h
index c36de2bdc6b73074a3bb943a5e4d6bec4b745aa0..34586cffddf6f33ddd5c98ab988c709330bc9278 100644
--- a/chrome/browser/ui/ash/launcher/chrome_launcher_controller_mus.h
+++ b/chrome/browser/ui/ash/launcher/chrome_launcher_controller_mus.h
@@ -5,15 +5,11 @@
#ifndef CHROME_BROWSER_UI_ASH_LAUNCHER_CHROME_LAUNCHER_CONTROLLER_MUS_H_
#define CHROME_BROWSER_UI_ASH_LAUNCHER_CHROME_LAUNCHER_CONTROLLER_MUS_H_
-#include <map>
-#include <memory>
#include <string>
#include "base/macros.h"
#include "chrome/browser/ui/ash/launcher/chrome_launcher_controller.h"
-class ChromeShelfItemDelegate;
-
class ChromeLauncherControllerMus : public ChromeLauncherController {
public:
ChromeLauncherControllerMus();
@@ -54,8 +50,7 @@ class ChromeLauncherControllerMus : public ChromeLauncherController {
bool allow_minimize) override;
void ActiveUserChanged(const std::string& user_email) override;
void AdditionalUserAddedToSession(Profile* profile) override;
- ash::ShelfAppMenuItemList GetAppMenuItemsForTesting(
- const ash::ShelfItem& item) override;
+ MenuItemList GetAppMenuItemsForTesting(const ash::ShelfItem& item) override;
std::vector<content::WebContents*> GetV1ApplicationsFromAppId(
const std::string& app_id) override;
void ActivateShellApp(const std::string& app_id, int window_index) override;
@@ -89,9 +84,6 @@ class ChromeLauncherControllerMus : public ChromeLauncherController {
// Pin the items set in the current profile's preferences.
void PinAppsFromPrefs();
- std::map<std::string, std::unique_ptr<ChromeShelfItemDelegate>>
- app_id_to_item_delegate_;
-
DISALLOW_COPY_AND_ASSIGN(ChromeLauncherControllerMus);
};

Powered by Google App Engine
This is Rietveld 408576698