[Mac] Refactor for updating the download item

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 <stddef.h>
 
 #include "base/mac/bundle_locations.h"
 #include "base/strings/sys_string_conversions.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"
 #include "chrome/browser/ui/cocoa/download/download_shelf_mac.h"
 #import "chrome/browser/ui/cocoa/download/download_shelf_view_cocoa.h"
 #include "content/public/browser/download_item.h"
 #include "content/public/browser/download_manager.h"
 #import "third_party/google_toolbox_for_mac/src/AppKit/GTMNSAnimation+Duration.h"
 #import "ui/base/cocoa/hover_button.h"
+#import "ui/base/cocoa/nsview_additions.h"
 
 using content::DownloadItem;
 
 // Download shelf autoclose behavior:
 //
 // The download shelf autocloses if all of this is true:
 // 1) An item on the shelf has just been opened or removed.
 // 2) All remaining items on the shelf have been opened in the past.
 // 3) The mouse leaves the shelf and remains off the shelf for 5 seconds.
 //
@@ skipping 39 matching lines @@
 - (void)closed;
 - (void)maybeAutoCloseAfterDelay;
 - (void)scheduleAutoClose;
 - (void)cancelAutoClose;
 - (void)autoClose;
 - (void)viewFrameDidChange:(NSNotification*)notification;
 - (void)installTrackingArea;
 - (void)removeTrackingArea;
 - (void)willEnterFullscreen;
 - (void)didExitFullscreen;
-- (void)updateDownloadItemView;
 - (void)updateCloseButton;
 @end
 
 
 @implementation DownloadShelfController
 
 - (id)initWithBrowser:(Browser*)browser
        resizeDelegate:(id<ViewResizer>)resizeDelegate {
   if ((self = [super initWithNibName:@"DownloadShelf"
                               bundle:base::mac::FrameworkBundle()])) {
@@ skipping 378 matching lines @@
 }
 
 - (void)willEnterFullscreen {
   isFullscreen_ = YES;
   [self updateCloseButton];
 }
 
 - (void)didExitFullscreen {
   isFullscreen_ = NO;
   [self updateCloseButton];
-  [self updateDownloadItemView];
-}
-
-- (void)updateDownloadItemView {
   for (DownloadItemController* controller in downloadItemControllers_.get())
-    [controller updateDownloadItemView];
+    [[controller view] cr_recursivelySetNeedsDisplay:YES];
 }
 
 - (void)updateCloseButton {
   if (!barIsVisible_)
     return;
 
   NSRect selfBounds = [[self view] bounds];
   NSRect hoverFrame = [hoverCloseButton_ frame];
   NSRect bounds;
 
   if (isFullscreen_) {
     bounds = NSMakeRect(NSMinX(hoverFrame), 0,
                         selfBounds.size.width - NSMinX(hoverFrame),
                         selfBounds.size.height);
   } else {
     bounds.origin.x = NSMinX(hoverFrame);
     bounds.origin.y = NSMidY(hoverFrame) -
                       kHoverCloseButtonDefaultSize.height / 2.0;
     bounds.size = kHoverCloseButtonDefaultSize;
   }
 
   // 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