Implementation of a full screen app list and re-alphabetized switches

ui/app_list/views/app_list_view.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_APP_LIST_VIEWS_APP_LIST_VIEW_H_
 #define UI_APP_LIST_VIEWS_APP_LIST_VIEW_H_
 
 #include <memory>
 
 #include "base/callback.h"
@@ skipping 16 matching lines @@
 
 namespace test {
 class AppListViewTestApi;
 }
 
 // AppListView is the top-level view and controller of app list UI. It creates
 // and hosts a AppsGridView and passes AppListModel to it for display.
 class APP_LIST_EXPORT AppListView : public views::BubbleDialogDelegateView,
                                     public SpeechUIModelObserver {
  public:
+  const char* GetClassName() const override;

[sky, 2017/04/07 03:28:19]

Group overrides of the same type together (in other words move this to around
line 78ish). Also, style guide says constructor/destructor is before other
functions.

[newcomer, 2017/04/10 16:34:53]

Done.

+
   // Does not take ownership of |delegate|.
   explicit AppListView(AppListViewDelegate* delegate);
   ~AppListView() override;
 
-  // Initializes the widget.
+  // Initializes the widget with a frame.
   void InitAsBubble(gfx::NativeView parent, int initial_apps_page);
 
+  // Initializes the widget to appear frameless.
+  void InitAsFramelessWindow(gfx::NativeView parent,
+                             int initial_apps_page,
+                             gfx::Rect bounds);

[sky, 2017/04/07 03:28:19]

const gfx::Rect& bounds

[newcomer, 2017/04/10 16:34:53]

Done.

+
   void SetBubbleArrow(views::BubbleBorder::Arrow arrow);
 
   void SetAnchorPoint(const gfx::Point& anchor_point);
 
   // If |drag_and_drop_host| is not NULL it will be called upon drag and drop
   // operations outside the application list. This has to be called after
   // InitAsBubble was called since the app list object needs to exist so that
   // it can set the host.
   void SetDragAndDropHostOfCurrentAppList(
       ApplicationDragAndDropHost* drag_and_drop_host);
@@ skipping 74 matching lines @@
 
   // For UMA and testing. If non-null, triggered when the app list is painted.
   base::Closure next_paint_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(AppListView);
 };
 
 }  // namespace app_list
 
 #endif  // UI_APP_LIST_VIEWS_APP_LIST_VIEW_H_