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

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

Issue 302803002: Refactor app list so AppsGridView owns the PaginationModel. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix tests and bugs. Created 6 years, 7 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 270f0eab2653ec45980f83e57f1733578e855681..0619fe9cc714fe7415a4e72c3dcfb6439a18816c 100644
--- a/ui/app_list/views/contents_view.h
+++ b/ui/app_list/views/contents_view.h
@@ -45,7 +45,6 @@ class APP_LIST_EXPORT ContentsView : public views::View {
};
ContentsView(AppListMainView* app_list_main_view,
- PaginationModel* pagination_model,
AppListModel* model,
AppListViewDelegate* view_delegate);
virtual ~ContentsView();
@@ -89,7 +88,7 @@ class APP_LIST_EXPORT ContentsView : public views::View {
virtual void OnScrollEvent(ui::ScrollEvent* event) OVERRIDE;
ShowState show_state_;
- PaginationModel* pagination_model_; // Owned by AppListController.
+ PaginationModel* pagination_model_; // Owned by AppListGridView.
xiyuan 2014/05/30 15:47:47 AppListGridView -> AppsGridView
Matt Giuca 2014/06/02 07:11:23 Done.
AppsContainerView* apps_container_view_; // Owned by the views hierarchy.
StartPageView* start_page_view_; // Owned by the views hierarchy.

Powered by Google App Engine
This is Rietveld 408576698