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

Unified Diff: chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc

Issue 2090133003: Moved methods that don't rely on internal state out of ChromeLauncherController. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nit changes Created 4 years, 6 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
« no previous file with comments | « chrome/browser/ui/ash/launcher/launcher_context_menu.cc ('k') | chrome/chrome_browser_ui.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc
diff --git a/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc b/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc
index 2bebb9ce09fa2cf5be1a804eb9e1a19fe25b76c9..a2e406b417a1f45966b21719cca2b2c81b6d62c6 100644
--- a/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc
+++ b/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.cc
@@ -23,7 +23,7 @@
#if defined(USE_ASH)
#include "ash/shelf/shelf_delegate.h"
#include "ash/shell.h"
-#include "chrome/browser/ui/ash/launcher/chrome_launcher_controller.h"
+#include "chrome/browser/ui/ash/launcher/chrome_launcher_controller_util.h"
#endif
AppInfoFooterPanel::AppInfoFooterPanel(gfx::NativeWindow parent_window,
@@ -171,9 +171,8 @@ bool AppInfoFooterPanel::CanSetPinnedToShelf() const {
// The Chrome app can't be unpinned, and extensions can't be pinned.
return app_->id() != extension_misc::kChromeAppId && !app_->is_extension() &&
- (!ChromeLauncherController::instance() ||
- ChromeLauncherController::instance()->GetPinnable(app_->id()) ==
- AppListControllerDelegate::PIN_EDITABLE);
+ (GetPinnableForAppID(app_->id(), profile_) ==
+ AppListControllerDelegate::PIN_EDITABLE);
}
#endif // USE_ASH
« no previous file with comments | « chrome/browser/ui/ash/launcher/launcher_context_menu.cc ('k') | chrome/chrome_browser_ui.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698