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

Unified Diff: chrome/browser/ui/cocoa/intents/web_intent_extension_prompt_view_controller.mm

Issue 11230023: Web Intents Mac: Add view animation (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix test Created 8 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/ui/cocoa/intents/web_intent_extension_prompt_view_controller.mm
diff --git a/chrome/browser/ui/cocoa/intents/web_intent_extension_prompt_view_controller.mm b/chrome/browser/ui/cocoa/intents/web_intent_extension_prompt_view_controller.mm
index 428242ed8038283ef338231a12fce3cfde81ce9e..6b087365210cc45b06792f181061df008c811f55 100644
--- a/chrome/browser/ui/cocoa/intents/web_intent_extension_prompt_view_controller.mm
+++ b/chrome/browser/ui/cocoa/intents/web_intent_extension_prompt_view_controller.mm
@@ -33,11 +33,6 @@
[[self view] addSubview:[viewController_ view]];
}
-- (void)clear {
- [[viewController_ view] removeFromSuperview];
- viewController_.reset();
-}
-
- (ExtensionInstallViewController*)viewController {
return viewController_;
}
@@ -53,4 +48,9 @@
[[viewController_ view] setFrameOrigin:NSZeroPoint];
}
+- (void)viewRemovedFromSuperview {
+ [[viewController_ view] removeFromSuperview];
+ viewController_.reset();
+}
+
@end

Powered by Google App Engine
This is Rietveld 408576698