Add a card background to the app list search result groups.

ui/app_list/views/search_result_view.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/app_list/views/search_result_view.h"
 
 #include <algorithm>
 
 #include "ui/app_list/app_list_constants.h"
+#include "ui/app_list/app_list_switches.h"
 #include "ui/app_list/search_result.h"
 #include "ui/app_list/views/progress_bar_view.h"
 #include "ui/app_list/views/search_result_actions_view.h"
 #include "ui/app_list/views/search_result_list_view.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/font.h"
 #include "ui/gfx/image/image_skia_operations.h"
 #include "ui/gfx/render_text.h"
 #include "ui/views/controls/button/image_button.h"
 #include "ui/views/controls/image_view.h"
@@ skipping 182 matching lines @@
 void SearchResultView::ChildPreferredSizeChanged(views::View* child) {
   Layout();
 }
 
 void SearchResultView::OnPaint(gfx::Canvas* canvas) {
   gfx::Rect rect(GetContentsBounds());
   if (rect.IsEmpty())
     return;
 
   gfx::Rect content_rect(rect);
-  content_rect.set_height(rect.height() - kBorderSize);
+  if (!switches::IsExperimentalAppListEnabled())

[Matt Giuca, 2014/11/13 00:17:23]

I'm curious about what effect this change, and the one below it, has on the
classic app list. Can you post screenshots, if there is a visible difference. Or
explain why not otherwise.

[calamity, 2014/11/13 03:55:31]

These have no effect on the classic app list. In the experimental app list, they
remove the separator line and remove the grey background respectively.

[Matt Giuca, 2014/11/13 07:10:41]

Acknowledged.

+    content_rect.set_height(rect.height() - kBorderSize);
 
   const bool selected = list_view_->IsResultViewSelected(this);
   const bool hover = state() == STATE_HOVERED || state() == STATE_PRESSED;
   if (selected)
     canvas->FillRect(content_rect, kSelectedColor);
   else if (hover)
     canvas->FillRect(content_rect, kHighlightedColor);
-  else
+  else if (!switches::IsExperimentalAppListEnabled())
     canvas->FillRect(content_rect, kContentsBackgroundColor);
 
   gfx::Rect border_bottom = gfx::SubtractRects(rect, content_rect);
   canvas->FillRect(border_bottom, kResultBorderColor);
 
   gfx::Rect text_bounds(rect);
   text_bounds.set_x(GetIconViewWidth());
   if (actions_view_->visible()) {
     text_bounds.set_width(
         rect.width() - GetIconViewWidth() - kTextTrailPadding -
@@ skipping 118 matching lines @@
   if (context_menu_runner_->RunMenuAt(GetWidget(),
                                       NULL,
                                       gfx::Rect(point, gfx::Size()),
                                       views::MENU_ANCHOR_TOPLEFT,
                                       source_type) ==
       views::MenuRunner::MENU_DELETED)
     return;
 }
 
 }  // namespace app_list