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

Unified Diff: ui/app_list/views/search_result_page_view.cc

Issue 896633005: Remove separate SearchResultListView from ContentsView. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@separate_start_page_tile_container
Patch Set: insert_newline Created 5 years, 10 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
« no previous file with comments | « ui/app_list/views/contents_view.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/app_list/views/search_result_page_view.cc
diff --git a/ui/app_list/views/search_result_page_view.cc b/ui/app_list/views/search_result_page_view.cc
index a923ba95139be4cf40bc3d2315a8a03ddbf1e3cc..fca91384a5d11ecf1c0abf90b3ce30d4c99f4a32 100644
--- a/ui/app_list/views/search_result_page_view.cc
+++ b/ui/app_list/views/search_result_page_view.cc
@@ -5,6 +5,7 @@
#include "ui/app_list/views/search_result_page_view.h"
#include "ui/app_list/app_list_constants.h"
+#include "ui/app_list/app_list_switches.h"
#include "ui/app_list/app_list_view_delegate.h"
#include "ui/app_list/views/app_list_main_view.h"
#include "ui/app_list/views/search_result_list_view.h"
@@ -45,9 +46,13 @@ class SearchCardView : public views::View {
} // namespace
SearchResultPageView::SearchResultPageView() : selected_index_(0) {
- SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical,
- kExperimentalWindowPadding, kTopPadding,
- kGroupSpacing));
+ if (switches::IsExperimentalAppListEnabled()) {
+ SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical,
+ kExperimentalWindowPadding,
+ kTopPadding, kGroupSpacing));
+ } else {
+ SetLayoutManager(new views::FillLayout);
+ }
}
SearchResultPageView::~SearchResultPageView() {
@@ -56,7 +61,11 @@ SearchResultPageView::~SearchResultPageView() {
void SearchResultPageView::AddSearchResultContainerView(
AppListModel::SearchResults* results_model,
SearchResultContainerView* result_container) {
- AddChildView(new SearchCardView(result_container));
+ views::View* view_to_add = result_container;
+ if (switches::IsExperimentalAppListEnabled())
+ view_to_add = new SearchCardView(result_container);
+
+ AddChildView(view_to_add);
result_container_views_.push_back(result_container);
result_container->SetResults(results_model);
}
« no previous file with comments | « ui/app_list/views/contents_view.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698