Remove 1 pixel padding around app list main view.

ui/app_list/views/apps_grid_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/apps_grid_view.h"
 
 #include <algorithm>
 #include <set>
 #include <string>
 
@@ skipping 42 matching lines @@
 
 namespace {
 
 // Distance a drag needs to be from the app grid to be considered 'outside', at
 // which point we rearrange the apps to their pre-drag configuration, as a drop
 // then would be canceled. We have a buffer to make it easier to drag apps to
 // other pages.
 const int kDragBufferPx = 20;
 
 // Padding space in pixels for fixed layout.
-const int kLeftRightPadding = 23;
 const int kTopPadding = 1;
+const int kBottomPadding = 1;
 
 // Padding space in pixels between pages.
 const int kPagePadding = 40;
 
 // Preferred tile size when showing in fixed layout.
 const int kPreferredTileWidth = 88;
 const int kPreferredTileHeight = 98;
 
 const int kExperimentalPreferredTileWidth = 90;
-const int kExperimentalPrefferedTileHeight = 90;
+const int kExperimentalPreferredTileHeight = 90;

[Matt Giuca, 2014/10/23 05:16:34]

lol.

 
 // Padding on each side of a tile.
 const int kExperimentalTileLeftRightPadding = 15;
 const int kExperimentalTileTopBottomPadding = 11;
 
 // Width in pixels of the area on the sides that triggers a page flip.
 const int kPageFlipZoneSize = 40;
 
 // Delay in milliseconds to do the page flip.
 const int kPageFlipDelayInMs = 1000;
@@ skipping 14 matching lines @@
 const int kFolderItemReparentDelay = 50;
 
 // Radius of the circle, in which if entered, show folder dropping preview
 // UI.
 const int kFolderDroppingCircleRadius = 39;
 
 // Returns the size of a tile view excluding its padding.
 gfx::Size GetTileViewSize() {
   return switches::IsExperimentalAppListEnabled()
              ? gfx::Size(kExperimentalPreferredTileWidth,
-                         kExperimentalPrefferedTileHeight)
+                         kExperimentalPreferredTileHeight)
              : gfx::Size(kPreferredTileWidth, kPreferredTileHeight);
 }
 
 // Returns the size of a tile view inccluding its padding.
 gfx::Size GetTotalTileSize() {
   gfx::Size size = GetTileViewSize();
   if (switches::IsExperimentalAppListEnabled())
     size.Enlarge(2 * kExperimentalTileLeftRightPadding,
                  2 * kExperimentalTileTopBottomPadding);
   return size;
@@ skipping 299 matching lines @@
   view_model_.Clear();
   RemoveAllChildViews(true);
 }
 
 void AppsGridView::SetLayout(int cols, int rows_per_page) {
   cols_ = cols;
   rows_per_page_ = rows_per_page;
 
   SetBorder(views::Border::CreateEmptyBorder(
       switches::IsExperimentalAppListEnabled() ? 0 : kTopPadding,
-      kLeftRightPadding,
-      0,
-      kLeftRightPadding));
+      kExperimentalWindowPadding,

[Matt Giuca, 2014/10/23 05:16:34]

This seems to be that the old app list will use kExperimentalWindowPadding here.
It might be simpler and clearer to just have a separate SetBorder call for old
and new app list.

[calamity, 2014/10/23 05:50:35]

Hmm. Fair enough. Done.

+      kBottomPadding,
+      kExperimentalWindowPadding));
 }
 
 void AppsGridView::ResetForShowApps() {
   activated_folder_item_view_ = NULL;
   ClearDragState();
   layer()->SetOpacity(1.0f);
   SetVisible(true);
   // Set all views to visible in case they weren't made visible again by an
   // incomplete animation.
   for (int i = 0; i < view_model_.view_size(); ++i) {
@@ skipping 1709 matching lines @@
 
 void AppsGridView::SetAsFolderDroppingTarget(const Index& target_index,
                                              bool is_target_folder) {
   AppListItemView* target_view =
       GetViewDisplayedAtSlotOnCurrentPage(target_index.slot);
   if (target_view)
     target_view->SetAsAttemptedFolderTarget(is_target_folder);
 }
 
 }  // namespace app_list