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

Unified Diff: chrome/browser/ui/ash/launcher/app_window_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_window_launcher_item_controller.cc
diff --git a/chrome/browser/ui/ash/launcher/app_window_launcher_item_controller.cc b/chrome/browser/ui/ash/launcher/app_window_launcher_item_controller.cc
index fa22849f1bb93f665595f9bfed54f49449933b8c..940bdb1ec0a58c73a30b4be9f86a1fb858bb8aca 100644
--- a/chrome/browser/ui/ash/launcher/app_window_launcher_item_controller.cc
+++ b/chrome/browser/ui/ash/launcher/app_window_launcher_item_controller.cc
@@ -69,15 +69,6 @@ void AppWindowLauncherItemController::SetActiveWindow(aura::Window* window) {
last_active_window_ = app_window;
}
-bool AppWindowLauncherItemController::IsVisible() const {
- // Return true if any windows are visible.
- for (const auto* window : windows_) {
- if (window->GetNativeWindow()->IsVisible())
- return true;
- }
- return false;
-}
-
void AppWindowLauncherItemController::Launch(ash::LaunchSource source,
int event_flags) {
launcher_controller()->LaunchApp(app_id(), source, ui::EF_NONE);

Powered by Google App Engine
This is Rietveld 408576698