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

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: 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/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..5fed727ad4e8a3357f041379c05d20ff75408792 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();
@@ -101,7 +100,7 @@ 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.
+ PaginationModel* pagination_model_; // Owned by AppListGridView.
xiyuan 2014/05/30 15:47:47 nit: AppListGridView -> AppsGridView
Matt Giuca 2014/06/02 07:11:23 Done.
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