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

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

Issue 302803002: Refactor app list so AppsGridView owns the PaginationModel. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Respond to review comments (tapted and xiyuan). 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/app_list_main_view.h
diff --git a/ui/app_list/views/app_list_main_view.h b/ui/app_list/views/app_list_main_view.h
index 366a138ce560c0eabd3174fbcb270792cb2acc7b..e3f3897c297a0069d61c31133a6463a493c54d2a 100644
--- a/ui/app_list/views/app_list_main_view.h
+++ b/ui/app_list/views/app_list_main_view.h
@@ -39,7 +39,6 @@ class APP_LIST_EXPORT AppListMainView : public views::View,
public:
// Takes ownership of |delegate|.
explicit AppListMainView(AppListViewDelegate* delegate,
- PaginationModel* pagination_model,
gfx::NativeView parent);
virtual ~AppListMainView();
@@ -74,6 +73,9 @@ class APP_LIST_EXPORT AppListMainView : public views::View,
void AddContentsView();
+ // Gets the PaginationModel owned by the AppsGridView.
+ PaginationModel* GetAppsPaginationModel();
+
// Loads icon image for the apps in the selected page of |pagination_model_|.
// |parent| is used to determine the image scale factor to use.
void PreloadIcons(gfx::NativeView parent);
@@ -101,7 +103,6 @@ class APP_LIST_EXPORT AppListMainView : public views::View,
virtual void OnResultUninstalled(SearchResult* result) OVERRIDE;
AppListViewDelegate* delegate_; // Owned by parent view (AppListView).
- PaginationModel* pagination_model_; // Owned by AppListController.
AppListModel* model_; // Unowned; ownership is handled by |delegate_|.
SearchBoxView* search_box_view_; // Owned by views hierarchy.

Powered by Google App Engine
This is Rietveld 408576698