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

Unified Diff: chrome/browser/ui/ash/launcher/browser_shortcut_launcher_item_controller.cc

Issue 2487953003: Remove LauncherItemController::IsOpen, check ShelfItem::status. (Closed)
Patch Set: Address comments. Created 4 years, 1 month 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/browser_shortcut_launcher_item_controller.cc
diff --git a/chrome/browser/ui/ash/launcher/browser_shortcut_launcher_item_controller.cc b/chrome/browser/ui/ash/launcher/browser_shortcut_launcher_item_controller.cc
index 90698773834ca0faf745f52812d9cd9d50430ce7..5f4e7d79cfe1ce79f04f53f7e437160f04f104bb 100644
--- a/chrome/browser/ui/ash/launcher/browser_shortcut_launcher_item_controller.cc
+++ b/chrome/browser/ui/ash/launcher/browser_shortcut_launcher_item_controller.cc
@@ -130,16 +130,6 @@ void BrowserShortcutLauncherItemController::SetShelfIDForBrowserWindowContents(
launcher_controller()->GetShelfIDForWebContents(web_contents));
}
-bool BrowserShortcutLauncherItemController::IsOpen() const {
- const BrowserList* browser_list = BrowserList::GetInstance();
- for (BrowserList::const_iterator it = browser_list->begin();
- it != browser_list->end(); ++it) {
- if (IsBrowserFromActiveUser(*it))
- return true;
- }
- return false;
-}
-
bool BrowserShortcutLauncherItemController::IsVisible() const {
Browser* last_browser =
chrome::FindTabbedBrowser(launcher_controller()->profile(), true);

Powered by Google App Engine
This is Rietveld 408576698