Clean up download item look, part 1 of many.

chrome/browser/cocoa/download_shelf_controller.mm

 // Copyright (c) 2009 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.
 
 #import "chrome/browser/cocoa/download_shelf_controller.h"
 
 #include "app/l10n_util.h"
 #include "base/mac_util.h"
 #include "base/sys_string_conversions.h"
 #include "chrome/browser/browser.h"
 #import "chrome/browser/cocoa/browser_window_controller.h"
 #include "chrome/browser/cocoa/browser_window_cocoa.h"
 #include "chrome/browser/cocoa/download_item_controller.h"
 #include "chrome/browser/cocoa/download_shelf_mac.h"
 #import "chrome/browser/cocoa/download_shelf_view.h"
 #include "grit/generated_resources.h"
 
 namespace {
 
 // Max number of download views we'll contain. Any time a view is added and
 // we already have this many download views, one is removed.
 const size_t kMaxDownloadItemCount = 16;
 
-// Border padding of a download item.
-const int kDownloadItemBorderPadding = 3;
-
 // Horizontal padding between two download items.
-const int kDownloadItemPadding = 10;
+const int kDownloadItemPadding = 2;
 
 // Duration for the open-new-leftmost-item animation, in seconds.
 const NSTimeInterval kDownloadItemOpenDuration = 0.8;
 
 }  // namespace
 
 @interface DownloadShelfController(Private)
 - (void)applyContentAreaOffset:(BOOL)apply;
 - (void)showDownloadShelf:(BOOL)enable;
 - (void)resizeDownloadLinkToFit;
@@ skipping 185 matching lines @@
            name:NSViewFrameDidChangeNotification
          object:[controller view]];
   [[NSNotificationCenter defaultCenter]
     addObserver:controller
        selector:@selector(updateVisibility:)
            name:NSViewFrameDidChangeNotification
          object:itemContainerView_];
 
   // Start at width 0...
   NSSize size = [controller.get() preferredSize];
-  NSRect frame = NSMakeRect(0, kDownloadItemBorderPadding, 0, size.height);
+  NSRect frame = NSMakeRect(0, 0, 0, size.height);
   [[controller.get() view] setFrame:frame];
 
   // ...then animate in
   frame.size.width = size.width;
   [NSAnimationContext beginGrouping];
   [[NSAnimationContext currentContext] setDuration:kDownloadItemOpenDuration];
   [[[controller.get() view] animator] setFrame:frame];
   [NSAnimationContext endGrouping];
 
   // Keep only a limited number of items in the shelf.
   if ([downloadItemControllers_ count] > kMaxDownloadItemCount) {
     DCHECK(kMaxDownloadItemCount > 0);
 
     // Since no user will ever see the item being removed (needs a horizontal
     // screen resolution greater than 3200 at 16 items at 200 pixels each),
     // there's no point in animating the removal.
     [self remove:[downloadItemControllers_ lastObject]];
   }
 
   // Finally, move the remaining items to the right. Skip the first item when
   // laying out the items, so that the longer animation duration we set up above
   // is not overwritten.
   [self layoutItems:YES];
 }
 
 @end