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

ui/app_list/views/app_list_background.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/app_list_background.h"
 
 #include "base/command_line.h"
 #include "third_party/skia/include/core/SkPaint.h"
 #include "third_party/skia/include/core/SkPath.h"
 #include "ui/app_list/app_list_constants.h"
@@ skipping 69 matching lines @@
 
   paint.setColor(kContentsBackgroundColor);
   canvas->DrawRect(contents_rect, paint);
 
   if (app_list::switches::IsExperimentalAppListEnabled()) {
     if (main_view_->visible()) {
       views::View* contents_view = main_view_->contents_view();
       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.

     }
   }
 
   canvas->Restore();
 }
 
 }  // namespace app_list