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

Unified Diff: ui/app_list/views/app_list_view.h

Issue 2802903003: Implementation of a full screen app list and re-alphabetized switches (Closed)
Patch Set: fixed some formatting/whitespace errors. Created 3 years, 8 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: ui/app_list/views/app_list_view.h
diff --git a/ui/app_list/views/app_list_view.h b/ui/app_list/views/app_list_view.h
index fb3b061ebef8785d33a8e979000d097a1cbf251c..bdff0b54982cede682e398f6a707a3cde2213fa3 100644
--- a/ui/app_list/views/app_list_view.h
+++ b/ui/app_list/views/app_list_view.h
@@ -35,10 +35,17 @@ class APP_LIST_EXPORT AppListView : public views::BubbleDialogDelegateView,
public SpeechUIModelObserver {
public:
// Does not take ownership of |delegate|.
- explicit AppListView(AppListViewDelegate* delegate);
+ explicit AppListView(AppListViewDelegate *delegate);
~AppListView() override;
- // Initializes the widget.
+ // Initialized the widget with a frame or a frameless view depending on
vadimt 2017/04/10 19:40:10 Initializes
newcomer 2017/04/13 21:13:59 fixed
+ // command line switches
+ void InitializeWindow(gfx::NativeView parent, int initial_apps_page,
vadimt 2017/04/10 19:40:10 Just "Initialize"?
newcomer 2017/04/13 21:13:59 Done.
+ const gfx::Point &center_of_display_window,
+ const gfx::Rect display_work_area_bounds);
+
+ // Initializes the widget as a bubble. Integrated into InitializeWindow but kept around for it's
vadimt 2017/04/10 19:40:10 Too long line? Please check with the style guide.
newcomer 2017/04/13 21:13:59 It now comply's with the 80 character per line lim
+ // usage in app_list_presenter_delegate_mus.cc
void InitAsBubble(gfx::NativeView parent, int initial_apps_page);
void SetBubbleArrow(views::BubbleBorder::Arrow arrow);
@@ -69,6 +76,7 @@ class APP_LIST_EXPORT AppListView : public views::BubbleDialogDelegateView,
// Overridden from views::View:
gfx::Size GetPreferredSize() const override;
void OnPaint(gfx::Canvas* canvas) override;
+ const char* GetClassName() const override;
// WidgetDelegate overrides:
bool ShouldHandleSystemCommands() const override;

Powered by Google App Engine
This is Rietveld 408576698