Fix the URL status bubble overlapping the Mac download shelf

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"
@@ skipping 159 matching lines @@
 
 - (BOOL)isVisible {
   return barIsVisible_;
 }
 
 - (void)show:(id)sender {
   [self showDownloadShelf:YES];
 }
 
 - (void)hide:(id)sender {
-  [self showDownloadShelf:NO];
+  // If |sender| isn't nil, then we're being closed from the UI by the user and
+  // we need to tell our shelf implementation to close. Otherwise, we're being
+  // closed programmatically by our shelf implementation.
+  if (sender)
+    bridge_->Close();
+  else
+    [self showDownloadShelf:NO];
+}
+
+- (float)height {
+  return shelfHeight_;
 }
 
 - (void)addDownloadItem:(BaseDownloadItemModel*)model {
   // TODO(thakis): we need to delete these at some point. There's no explicit
   // mass delete on windows, figure out where they do it.
 
   // TODO(thakis): RTL support?
   // (l10n_util::GetTextDirection() == l10n_util::RIGHT_TO_LEFT)
   int startX = kDownloadItemBorderPadding +
                (kDownloadItemWidth + kDownloadItemPadding) *
                [downloadItemControllers_ count];
 
   NSRect position = NSMakeRect(startX, kDownloadItemBorderPadding,
                                kDownloadItemWidth, kDownloadItemHeight);
   scoped_nsobject<DownloadItemController> controller(
       [[DownloadItemController alloc] initWithFrame:position
                                               model:model
                                               shelf:self]);
   [downloadItemControllers_ addObject:controller.get()];
 
   [[self view] addSubview:[controller.get() view]];
 }
 
 @end