Experimental app list: Enforce a 24-pixel padding on the top and sides.

ui/app_list/views/app_list_background.cc

Index: ui/app_list/views/app_list_background.cc
diff --git a/ui/app_list/views/app_list_background.cc b/ui/app_list/views/app_list_background.cc
index 0857d5b9a2de7fb8542436de1f8ca765ea5e569e..bf33a38af08beb76128bab45191b60c4b8760763 100644
--- a/ui/app_list/views/app_list_background.cc
+++ b/ui/app_list/views/app_list_background.cc
@@ -87,15 +87,19 @@ void AppListBackground::Paint(gfx::Canvas* canvas,
       const gfx::Rect contents_view_view_bounds =
           contents_view->ConvertRectToWidget(contents_view->GetLocalBounds());
       gfx::Rect separator_rect(contents_rect);
+      separator_rect.set_x(kExperimentalWindowPadding);
+      separator_rect.set_width(contents_view_view_bounds.width() -
+                               kExperimentalWindowPadding * 2);

[calamity, 2014/09/19 05:10:56]

Use Rect::Inset()

[Matt Giuca, 2014/09/19 09:56:05]

Done. (Also this was buggy before because it assumes x was 0 beforehand --- it's
actually 5.)

       separator_rect.set_y(contents_view_view_bounds.bottom());
       separator_rect.set_height(kBottomSeparatorSize);
       canvas->FillRect(separator_rect, kBottomSeparatorColor);
+
       int contents_switcher_top = separator_rect.bottom();
       gfx::Rect contents_switcher_rect(bounds.x(),
                                        contents_switcher_top,
                                        bounds.width(),
                                        bounds.bottom() - contents_switcher_top);
-      paint.setColor(kContentsSwitcherBackgroundColor);
+      paint.setColor(kContentsBackgroundColor);
       canvas->DrawRect(contents_switcher_rect, paint);

[calamity, 2014/09/19 05:10:56]

96-103 is unnecessary now.

[Matt Giuca, 2014/09/19 09:56:05]

Done.

     }
   }