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

Side by Side Diff: chrome/browser/ui/ash/launcher/arc_app_deferred_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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/ash/launcher/arc_app_deferred_launcher_item_controll er.h" 5 #include "chrome/browser/ui/ash/launcher/arc_app_deferred_launcher_item_controll er.h"
6 6
7 #include "base/strings/utf_string_conversions.h" 7 #include "base/strings/utf_string_conversions.h"
8 #include "chrome/browser/profiles/profile.h" 8 #include "chrome/browser/profiles/profile.h"
9 #include "chrome/browser/ui/app_list/arc/arc_app_list_prefs.h" 9 #include "chrome/browser/ui/app_list/arc/arc_app_list_prefs.h"
10 #include "chrome/browser/ui/ash/launcher/arc_app_deferred_launcher_controller.h" 10 #include "chrome/browser/ui/ash/launcher/arc_app_deferred_launcher_controller.h"
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
62 62
63 bool ArcAppDeferredLauncherItemController::ShouldShowTooltip() { 63 bool ArcAppDeferredLauncherItemController::ShouldShowTooltip() {
64 return true; 64 return true;
65 } 65 }
66 66
67 void ArcAppDeferredLauncherItemController::Close() { 67 void ArcAppDeferredLauncherItemController::Close() {
68 if (host_) 68 if (host_)
69 host_->Close(app_id()); 69 host_->Close(app_id());
70 } 70 }
71 71
72 bool ArcAppDeferredLauncherItemController::IsOpen() const {
73 return true;
74 }
75
76 bool ArcAppDeferredLauncherItemController::IsVisible() const { 72 bool ArcAppDeferredLauncherItemController::IsVisible() const {
77 return true; 73 return true;
78 } 74 }
79 75
80 void ArcAppDeferredLauncherItemController::Launch(ash::LaunchSource source, 76 void ArcAppDeferredLauncherItemController::Launch(ash::LaunchSource source,
81 int event_flags) {} 77 int event_flags) {}
82 78
83 ash::ShelfItemDelegate::PerformedAction 79 ash::ShelfItemDelegate::PerformedAction
84 ArcAppDeferredLauncherItemController::Activate(ash::LaunchSource source) { 80 ArcAppDeferredLauncherItemController::Activate(ash::LaunchSource source) {
85 return ash::ShelfItemDelegate::kNoAction; 81 return ash::ShelfItemDelegate::kNoAction;
86 } 82 }
87 83
88 ChromeLauncherAppMenuItems 84 ChromeLauncherAppMenuItems
89 ArcAppDeferredLauncherItemController::GetApplicationList(int event_flags) { 85 ArcAppDeferredLauncherItemController::GetApplicationList(int event_flags) {
90 return ChromeLauncherAppMenuItems(); 86 return ChromeLauncherAppMenuItems();
91 } 87 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698