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

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

Issue 316393002: App list uses PaginationModel to transition between pages. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed tests -- adds blank pages so we have 3 pages in the test environment. Created 6 years, 6 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/contents_view.h
diff --git a/ui/app_list/views/contents_view.h b/ui/app_list/views/contents_view.h
index c9dfa0bcd3f56568eaa025bcd7e8e5c5d3991a5d..20906743006f4853c7b5d3587f5c1206d34fc1f5 100644
--- a/ui/app_list/views/contents_view.h
+++ b/ui/app_list/views/contents_view.h
@@ -11,10 +11,11 @@
#include "base/compiler_specific.h"
#include "base/memory/scoped_ptr.h"
#include "ui/app_list/app_list_export.h"
+#include "ui/app_list/pagination_model.h"
+#include "ui/app_list/pagination_model_observer.h"
#include "ui/views/view.h"
namespace views {
-class BoundsAnimator;
class ViewModel;
}
@@ -36,7 +37,8 @@ class StartPageView;
// one of which can be active at a given time. ContentsView provides the user
// interface for switching between launcher pages, and animates the transition
// between them.
-class APP_LIST_EXPORT ContentsView : public views::View {
+class APP_LIST_EXPORT ContentsView : public views::View,
+ public PaginationModelObserver {
public:
// Values of this enum denote special launcher pages that require hard-coding.
// Launcher pages are not required to have a NamedPage enum value.
@@ -67,7 +69,7 @@ class APP_LIST_EXPORT ContentsView : public views::View {
void SetActivePage(int page_index);
// The index of the currently active launcher page.
- int active_page_index() const { return active_page_; }
+ int GetActivePageIndex() const;
// True if |named_page| is the current active laucher page.
bool IsNamedPageActive(NamedPage named_page) const;
@@ -75,11 +77,17 @@ class APP_LIST_EXPORT ContentsView : public views::View {
// Gets the index of a launcher page in |view_model_|, by NamedPage.
int GetPageIndexForNamedPage(NamedPage named_page) const;
+ int NumLauncherPages() const;
+
void Prerender();
AppsContainerView* apps_container_view() { return apps_container_view_; }
StartPageView* start_page_view() { return start_page_view_; }
SearchResultListView* search_results_view() { return search_results_view_; }
+ views::View* GetPageView(int index);
+
+ // Adds a blank launcher page. For use in tests only.
+ void AddBlankPageForTesting();
// Overridden from views::View:
virtual gfx::Size GetPreferredSize() const OVERRIDE;
@@ -87,12 +95,19 @@ class APP_LIST_EXPORT ContentsView : public views::View {
virtual bool OnKeyPressed(const ui::KeyEvent& event) OVERRIDE;
virtual bool OnMouseWheel(const ui::MouseWheelEvent& event) OVERRIDE;
+ // Overridden from PaginationModelObserver:
+ virtual void TotalPagesChanged() OVERRIDE;
+ virtual void SelectedPageChanged(int old_selected, int new_selected) OVERRIDE;
+ virtual void TransitionStarted() OVERRIDE;
+ virtual void TransitionChanged() OVERRIDE;
+
private:
// Invoked when active view is changed.
void ActivePageChanged();
- void CalculateIdealBounds();
- void AnimateToIdealBounds();
+ // Calculates and sets the bounds for the subviews. If there is currently an
+ // animation, this positions the views as appropriate for the current frame.
+ void UpdatePageBounds();
// Adds |view| as a new page to the end of the list of launcher pages. The
// view is inserted as a child of the ContentsView. There is no name
@@ -105,15 +120,14 @@ class APP_LIST_EXPORT ContentsView : public views::View {
int AddLauncherPage(views::View* view, NamedPage named_page);
// Gets the PaginationModel owned by the AppsGridView.
+ // Note: This is different to |pagination_model_|, which manages top-level
+ // launcher-page pagination.
PaginationModel* GetAppsPaginationModel();
// Overridden from ui::EventHandler:
virtual void OnGestureEvent(ui::GestureEvent* event) OVERRIDE;
virtual void OnScrollEvent(ui::ScrollEvent* event) OVERRIDE;
- // Index into |view_model_| of the currently active page.
- int active_page_;
-
// Special sub views of the ContentsView. All owned by the views hierarchy.
AppsContainerView* apps_container_view_;
SearchResultListView* search_results_view_;
@@ -124,7 +138,9 @@ class APP_LIST_EXPORT ContentsView : public views::View {
scoped_ptr<views::ViewModel> view_model_;
// Maps NamedPage onto |view_model_| indices.
std::map<NamedPage, int> named_page_to_view_;
- scoped_ptr<views::BoundsAnimator> bounds_animator_;
+
+ // Manages the pagination for the launcher pages.
+ PaginationModel pagination_model_;
DISALLOW_COPY_AND_ASSIGN(ContentsView);
};

Powered by Google App Engine
This is Rietveld 408576698