Refactor app list ContentsView to use pointer members to find views.

ui/app_list/views/contents_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_CONTENTS_VIEW_H_
 #define UI_APP_LIST_VIEWS_CONTENTS_VIEW_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 12 matching lines @@
 namespace app_list {
 
 class AppsGridView;
 class ApplicationDragAndDropHost;
 class AppListFolderItem;
 class AppListMainView;
 class AppListModel;
 class AppListViewDelegate;
 class AppsContainerView;
 class PaginationModel;
+class SearchResultListView;
 class StartPageView;
 
 // A view to manage sub views under the search box (apps grid view + page
 // switcher and search results). The two sets of sub views are mutually
 // exclusive. ContentsView manages a show state to choose one set to show
 // and animates the transition between show states.
 class APP_LIST_EXPORT ContentsView : public views::View {
  public:
   enum ShowState {
     SHOW_APPS,
@@ skipping 42 matching lines @@
 
   // Gets the PaginationModel owned by the AppsGridView.
   PaginationModel* GetAppsPaginationModel();
 
   // Overridden from ui::EventHandler:
   virtual void OnGestureEvent(ui::GestureEvent* event) OVERRIDE;
   virtual void OnScrollEvent(ui::ScrollEvent* event) OVERRIDE;
 
   ShowState show_state_;
 
-  AppsContainerView* apps_container_view_;  // Owned by the views hierarchy.
-  StartPageView* start_page_view_;          // Owned by the views hierarchy.
+  // Special sub views of the ContentsView. All owned by the views hierarchy.

[benwells, 2014/06/05 06:33:45]

How are they special??

[Matt Giuca, 2014/06/05 06:54:11]

"Special" means that we need hard-coded access to them from our code. At the
moment, they're all special (just like snowflakes). But once we start adding
dynamic pages, this will make more sense because they won't be considered
special.

+  AppsContainerView* apps_container_view_;
+  SearchResultListView* search_results_view_;
+  StartPageView* start_page_view_;
 
   AppListMainView* app_list_main_view_;     // Parent view, owns this.
 
   scoped_ptr<views::ViewModel> view_model_;
   scoped_ptr<views::BoundsAnimator> bounds_animator_;
 
   DISALLOW_COPY_AND_ASSIGN(ContentsView);
 };
 
 }  // namespace app_list
 
 #endif  // UI_APP_LIST_VIEWS_CONTENTS_VIEW_H_