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

Side by Side Diff: ui/app_list/presenter/app_list_presenter_impl.h

Issue 2434573004: Close app list widget when users click outside of the widget (Closed)
Patch Set: Add comments in destructor for clarification. 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 UI_APP_LIST_PRESENTER_APP_LIST_PRESENTER_IMPL_H_ 5 #ifndef UI_APP_LIST_PRESENTER_APP_LIST_PRESENTER_IMPL_H_
6 #define UI_APP_LIST_PRESENTER_APP_LIST_PRESENTER_IMPL_H_ 6 #define UI_APP_LIST_PRESENTER_APP_LIST_PRESENTER_IMPL_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 ~AppListPresenterImpl() override; 46 ~AppListPresenterImpl() override;
47 47
48 // Returns app list window or NULL if it is not visible. 48 // Returns app list window or NULL if it is not visible.
49 aura::Window* GetWindow(); 49 aura::Window* GetWindow();
50 50
51 // Returns app list view if one exists, or NULL otherwise. 51 // Returns app list view if one exists, or NULL otherwise.
52 AppListView* GetView() { return view_; } 52 AppListView* GetView() { return view_; }
53 53
54 // AppListPresenter: 54 // AppListPresenter:
55 void Show(int64_t display_id) override; 55 void Show(int64_t display_id) override;
56 void Dismiss() override; 56 void Dismiss() final;
57 void ToggleAppList(int64_t display_id) override; 57 void ToggleAppList(int64_t display_id) override;
58 bool IsVisible() const override; 58 bool IsVisible() const override;
59 bool GetTargetVisibility() const override; 59 bool GetTargetVisibility() const override;
60 60
61 private: 61 private:
62 friend class test::AppListPresenterImplTestApi; 62 friend class test::AppListPresenterImplTestApi;
63 63
64 // Sets the app list view and attempts to show it. 64 // Sets the app list view and attempts to show it.
65 void SetView(AppListView* view); 65 void SetView(AppListView* view);
66 66
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 112
113 // Whether should schedule snap back animation. 113 // Whether should schedule snap back animation.
114 bool should_snap_back_ = false; 114 bool should_snap_back_ = false;
115 115
116 DISALLOW_COPY_AND_ASSIGN(AppListPresenterImpl); 116 DISALLOW_COPY_AND_ASSIGN(AppListPresenterImpl);
117 }; 117 };
118 118
119 } // namespace app_list 119 } // namespace app_list
120 120
121 #endif // UI_APP_LIST_PRESENTER_APP_LIST_PRESENTER_IMPL_H_ 121 #endif // UI_APP_LIST_PRESENTER_APP_LIST_PRESENTER_IMPL_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/ash/app_list/app_list_service_ash.cc ('k') | ui/app_list/presenter/app_list_presenter_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698