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

Side by Side Diff: chrome/browser/ui/ash/launcher/arc_app_deferred_launcher_item_controller.h

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 #ifndef CHROME_BROWSER_UI_ASH_LAUNCHER_ARC_APP_DEFERRED_LAUNCHER_ITEM_CONTROLLER _H_ 5 #ifndef CHROME_BROWSER_UI_ASH_LAUNCHER_ARC_APP_DEFERRED_LAUNCHER_ITEM_CONTROLLER _H_
6 #define CHROME_BROWSER_UI_ASH_LAUNCHER_ARC_APP_DEFERRED_LAUNCHER_ITEM_CONTROLLER _H_ 6 #define CHROME_BROWSER_UI_ASH_LAUNCHER_ARC_APP_DEFERRED_LAUNCHER_ITEM_CONTROLLER _H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 19 matching lines...) Expand all
30 ash::ShelfItemDelegate::PerformedAction ItemSelected( 30 ash::ShelfItemDelegate::PerformedAction ItemSelected(
31 const ui::Event& event) override; 31 const ui::Event& event) override;
32 base::string16 GetTitle() override; 32 base::string16 GetTitle() override;
33 bool CanPin() const override; 33 bool CanPin() const override;
34 ash::ShelfMenuModel* CreateApplicationMenu(int event_flags) override; 34 ash::ShelfMenuModel* CreateApplicationMenu(int event_flags) override;
35 bool IsDraggable() override; 35 bool IsDraggable() override;
36 bool ShouldShowTooltip() override; 36 bool ShouldShowTooltip() override;
37 void Close() override; 37 void Close() override;
38 38
39 // LauncherItemController overrides: 39 // LauncherItemController overrides:
40 bool IsOpen() const override;
41 bool IsVisible() const override; 40 bool IsVisible() const override;
42 void Launch(ash::LaunchSource source, int event_flags) override; 41 void Launch(ash::LaunchSource source, int event_flags) override;
43 ash::ShelfItemDelegate::PerformedAction Activate( 42 ash::ShelfItemDelegate::PerformedAction Activate(
44 ash::LaunchSource source) override; 43 ash::LaunchSource source) override;
45 ChromeLauncherAppMenuItems GetApplicationList(int event_flags) override; 44 ChromeLauncherAppMenuItems GetApplicationList(int event_flags) override;
46 45
47 private: 46 private:
48 base::WeakPtr<ArcAppDeferredLauncherController> host_; 47 base::WeakPtr<ArcAppDeferredLauncherController> host_;
49 const base::Time start_time_; 48 const base::Time start_time_;
50 49
51 DISALLOW_COPY_AND_ASSIGN(ArcAppDeferredLauncherItemController); 50 DISALLOW_COPY_AND_ASSIGN(ArcAppDeferredLauncherItemController);
52 }; 51 };
53 52
54 #endif // CHROME_BROWSER_UI_ASH_LAUNCHER_ARC_APP_DEFERRED_LAUNCHER_ITEM_CONTROL LER_H_ 53 #endif // CHROME_BROWSER_UI_ASH_LAUNCHER_ARC_APP_DEFERRED_LAUNCHER_ITEM_CONTROL LER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698