Experimental app list: Added a current page indicator to the launcher.

ui/app_list/views/contents_view.cc

 // 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.
 
 #include "ui/app_list/views/contents_view.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "grit/ui_resources.h"
@@ skipping 58 matching lines @@
   }
 
   apps_container_view_ = new AppsContainerView(app_list_main_view_, model);
 
   int initial_page_index = AddLauncherPage(
       apps_container_view_, IDR_APP_LIST_APPS_ICON, NAMED_PAGE_APPS);
   if (app_list::switches::IsExperimentalAppListEnabled())
     initial_page_index = GetPageIndexForNamedPage(NAMED_PAGE_START);
 
   pagination_model_.SelectPage(initial_page_index, false);
+  if (contents_switcher_view_)
+    contents_switcher_view_->SelectedPageChanged(-1, initial_page_index);

[calamity, 2014/07/09 05:50:19]

Is this necessary? I would think the line above would trigger
ContentsView::SelectedPageChanged() since the observer is registered in the
constructor.

[kcarattini, 2014/07/09 06:45:22]

I would have thought so too, but it doesn't seem to work without it.

 
   // Needed to update the main search box visibility.
   ActivePageChanged(false);
 }
 
 void ContentsView::CancelDrag() {
   if (apps_container_view_->apps_grid_view()->has_dragged_view())
     apps_container_view_->apps_grid_view()->EndDrag(true);
   if (apps_container_view_->app_list_folder_view()
           ->items_grid_view()
@@ skipping 222 matching lines @@
     return true;
   }
 
   return false;
 }
 
 void ContentsView::TotalPagesChanged() {
 }
 
 void ContentsView::SelectedPageChanged(int old_selected, int new_selected) {
+  if (contents_switcher_view_)
+    contents_switcher_view_->SelectedPageChanged(old_selected, new_selected);
 }
 
 void ContentsView::TransitionStarted() {
 }
 
 void ContentsView::TransitionChanged() {
   UpdatePageBounds();
 }
 
 void ContentsView::OnGestureEvent(ui::GestureEvent* event) {
@@ skipping 47 matching lines @@
   if (std::abs(offset) > kMinScrollToSwitchPage) {
     if (!GetAppsPaginationModel()->has_transition()) {
       GetAppsPaginationModel()->SelectPageRelative(offset > 0 ? -1 : 1, true);
     }
     event->SetHandled();
     event->StopPropagation();
   }
 }
 
 }  // namespace app_list