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

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

Issue 2622423002: Remove unused LauncherItemController::IsVisible() (Closed)
Patch Set: Created 3 years, 11 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/app_shortcut_launcher_item_controller.cc
diff --git a/chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc b/chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc
index 8788a4506a06e41c4b7e3e6070ed55edf3329361..86b08fa8dee6c79e1b081e4389aa5aa216a8c67d 100644
--- a/chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc
+++ b/chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc
@@ -96,18 +96,6 @@ AppShortcutLauncherItemController::AppShortcutLauncherItemController(
AppShortcutLauncherItemController::~AppShortcutLauncherItemController() {
}
-bool AppShortcutLauncherItemController::IsVisible() const {
- // Return true if any browser window associated with the app is visible.
- std::vector<content::WebContents*> content =
- chrome_launcher_controller_->GetV1ApplicationsFromAppId(app_id());
- for (size_t i = 0; i < content.size(); i++) {
- Browser* browser = chrome::FindBrowserWithWebContents(content[i]);
- if (browser && browser->window()->GetNativeWindow()->IsVisible())
- return true;
- }
- return false;
-}
-
void AppShortcutLauncherItemController::Launch(ash::LaunchSource source,
int event_flags) {
// Launching an app replaces shortcut item controller to app controller. As

Powered by Google App Engine
This is Rietveld 408576698