Cleans up our autorelease handling so that we don't create a layered ...

chrome/browser/renderer_host/render_widget_host_view_mac.mm

 // Copyright (c) 2006-2008 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.
 
 #include "chrome/browser/renderer_host/render_widget_host_view_mac.h"
 
 #include "base/histogram.h"
 #include "base/scoped_nsobject.h"
 #include "base/string_util.h"
 #include "base/sys_string_conversions.h"
@@ skipping 271 matching lines @@
   // then the event causing the destroy had also cancelled any popups by the
   // time Destroy() was called. On the Mac we have to destroy all the popups
   // ourselves.
 
   // Depth-first destroy all popups. Use ShutdownHost() to enforce deepest-first
   // ordering.
   for (RenderWidgetHostViewCocoa* subview in [cocoa_view_ subviews]) {
     [subview renderWidgetHostViewMac]->ShutdownHost();
   }
 
-  // We've been told to destroy.
-  [cocoa_view_ retain];
-  [cocoa_view_ removeFromSuperview];
-  [cocoa_view_ autorelease];
-
   // We get this call just before |render_widget_host_| deletes
   // itself.  But we are owned by |cocoa_view_|, which may be retained
   // by some other code.  Examples are TabContentsViewMac's
   // |latent_focus_view_| and TabWindowController's
   // |cachedContentView_|.
   render_widget_host_ = NULL;
 }
 
 // Called from the renderer to tell us what the tooltip text should be. It
 // calls us frequently so we need to cache the value to prevent doing a lot
@@ skipping 230 matching lines @@
 }
 
 - (void)setIgnoreKeyEvents:(BOOL)ignorekeyEvents {
   ignoreKeyEvents_ = ignorekeyEvents;
 }
 
 - (BOOL)performKeyEquivalent:(NSEvent*)theEvent {
   if (ignoreKeyEvents_)
     return NO;
 
-  // We have some magic in |CrApplication sendEvent:| that always sends key 
+  // We have some magic in |CrApplication sendEvent:| that always sends key
   // events to |keyEvent:| so that cocoa doesn't have a chance to intercept it.
   DCHECK([[self window] firstResponder] != self);
   return NO;
 }
 
 - (void)keyEvent:(NSEvent*)theEvent {
   if (ignoreKeyEvents_)
     return;
 
   // TODO(avi): Possibly kill self? See RenderWidgetHostViewWin::OnKeyEvent and
@@ skipping 691 matching lines @@
     renderWidgetHostView_->render_widget_host_->ForwardKeyboardEvent(event);
   } else {
     renderWidgetHostView_->render_widget_host_->ImeConfirmComposition(
         UTF8ToUTF16([im_text UTF8String]));
   }
   renderWidgetHostView_->im_composing_ = false;
 }
 
 @end