Download shelf autohides on showing in shell, just same as regular open

chrome/browser/ui/cocoa/download/download_shelf_controller.mm

 // 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.
 
 #import "chrome/browser/ui/cocoa/download/download_shelf_controller.h"
 
 #include "base/mac/bundle_locations.h"
 #include "base/mac/mac_util.h"
 #include "base/strings/sys_string_conversions.h"
+#include "chrome/browser/download/download_item_model.h"
 #include "chrome/browser/download/download_stats.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/themes/theme_service.h"
 #include "chrome/browser/themes/theme_service_factory.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/chrome_pages.h"
 #import "chrome/browser/ui/cocoa/animatable_view.h"
 #include "chrome/browser/ui/cocoa/browser_window_cocoa.h"
 #import "chrome/browser/ui/cocoa/browser_window_controller.h"
 #include "chrome/browser/ui/cocoa/download/download_item_controller.h"
@@ skipping 188 matching lines @@
     [self maybeAutoCloseAfterDelay];
   }
 }
 
 - (void)downloadWasOpened:(DownloadItemController*)item_controller {
   // This should only be called on the main thead.
   DCHECK([NSThread isMainThread]);
   [self maybeAutoCloseAfterDelay];
 }
 
+- (void)downloadWasShown:(DownloadItemController*)item_controller {
+  // This should only be called on the main thead.
+  DCHECK([NSThread isMainThread]);
+  [self maybeAutoCloseAfterDelay];
+}
+
 // We need to explicitly release our download controllers here since they need
 // to remove themselves as observers before the remaining shutdown happens.
 - (void)exiting {
   [[self animatableView] stopAnimation];
   [self removeTrackingArea];
   [self cancelAutoClose];
   while ([downloadItemControllers_ count] > 0) {
     [self removeDownload:[downloadItemControllers_ lastObject]
           isShelfClosing:YES];
   }
@@ skipping 145 matching lines @@
         [downloadItemControllers_ objectAtIndex:i];
     DownloadItem* download = [itemController download];
     DownloadItem::DownloadState state = download->GetState();
     bool isTransferDone = state == DownloadItem::COMPLETE ||
                           state == DownloadItem::CANCELLED ||
                           state == DownloadItem::INTERRUPTED;
     if (isTransferDone && !download->IsDangerous()) {
       [self removeDownload:itemController
             isShelfClosing:YES];
     } else {
-      // Treat the item as opened when we close. This way if we get shown again
+      // Treat the item as 'acted' when we close. This way if we get shown again
       // the user need not open this item for the shelf to auto-close.
-      download->SetOpened(true);
+      DownloadItemModel(download).SetUserActed(true);
       ++i;
     }
   }
 }
 
 - (void)mouseEntered:(NSEvent*)event {
   isMouseInsideView_ = YES;
   // If the mouse re-enters the download shelf, cancel the auto-close.  Further
   // mouse exits should not trigger autoclose.
   if (shouldCloseOnMouseExit_) {
@@ skipping 24 matching lines @@
                                            selector:@selector(autoClose)
                                              object:nil];
 }
 
 - (void)maybeAutoCloseAfterDelay {
   // We can close the shelf automatically if all the downloads on the shelf have
   // been opened.
   for (NSUInteger i = 0; i < [downloadItemControllers_ count]; ++i) {
     DownloadItemController* itemController =
         [downloadItemControllers_ objectAtIndex:i];
-    if (![itemController download]->GetOpened())
+    DownloadItem* download = [itemController download];
+    if (!DownloadItemModel(download).GetUserActed())
       return;
   }
 
   if ([self isVisible] && [downloadItemControllers_ count] > 0 &&
       isMouseInsideView_) {
     // If there are download items on the shelf and the user is potentially stil
     // interacting with them, schedule an auto close after the user moves the
     // mouse off the shelf.
     shouldCloseOnMouseExit_ = YES;
   } else {
@@ skipping 61 matching lines @@
   }
 
   // Set the tracking off to create a new tracking area for the control.
   // When changing the bounds/frame on a HoverButton, the tracking isn't updated
   // correctly, it needs to be turned off and back on.
   [hoverCloseButton_ setTrackingEnabled:NO];
   [hoverCloseButton_ setFrame:bounds];
   [hoverCloseButton_ setTrackingEnabled:YES];
 }
 @end