Makes it so when NSViewControllers go away that they remove their views from ...

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 "app/resource_bundle.h"
 #include "base/mac_util.h"
 #include "base/sys_string_conversions.h"
@@ skipping 82 matching lines @@
   NSImage* favicon = rb.GetNSImageNamed(IDR_DOWNLOADS_FAVICON);
   DCHECK(favicon);
   [image_ setImage:favicon];
 
   [self resizeDownloadLinkToFit];
 }
 
 - (void)dealloc {
   // The controllers will unregister themselves as observers when they are
   // deallocated. No need to do that here.
+
+  // Remove our view from its superview so it doesn't attempt to reference
+  // it when the controller is gone.
+  [[self view] removeFromSuperview];
   [super dealloc];
 }
 
 - (void)resizeDownloadLinkToFit {
   // Get width required by localized download link text.
   // http://developer.apple.com/documentation/Cocoa/Conceptual/TextLayout/Tasks/StringHeight.html
   [[showAllDownloadsLink_ textContainer] setLineFragmentPadding:0.0];
   (void)[[showAllDownloadsLink_ layoutManager] glyphRangeForTextContainer:
       [showAllDownloadsLink_ textContainer]];
   NSRect textRect = [[showAllDownloadsLink_ layoutManager]
@@ skipping 172 matching lines @@
     if (isTransferDone &&
         [itemController download]->safety_state() != DownloadItem::DANGEROUS) {
       [self remove:itemController];
     } else {
       ++i;
     }
   }
 }
 
 @end