Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(225)

Unified Diff: chrome/browser/cocoa/browser_window_controller.mm

Issue 343024: Cleans up our autorelease handling so that we don't create a layered ... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/cocoa/browser_window_controller.mm
===================================================================
--- chrome/browser/cocoa/browser_window_controller.mm (revision 30592)
+++ chrome/browser/cocoa/browser_window_controller.mm (working copy)
@@ -262,13 +262,7 @@
// that its window has on our window before our window goes away.
delete statusBubble_;
statusBubble_ = NULL;
- // We can't actually use |-autorelease| here because there's an embedded
- // run loop in the |-performClose:| which contains its own autorelease pool.
- // Instead we call it after a zero-length delay, which gets us back
- // to the main event loop.
- [self performSelector:@selector(autorelease)
- withObject:nil
- afterDelay:0];
+ [self autorelease];
}
- (void)attachConstrainedWindow:(ConstrainedWindowMac*)window {

Powered by Google App Engine
This is Rietveld 408576698