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

Unified Diff: chrome/browser/ui/cocoa/extensions/media_galleries_dialog_cocoa.mm

Issue 10854150: Cocoa: Media gallery dialog unit test (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix gtk Created 8 years, 4 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/extensions/media_galleries_dialog_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/extensions/media_galleries_dialog_cocoa.mm b/chrome/browser/ui/cocoa/extensions/media_galleries_dialog_cocoa.mm
index 00aff4d4c656d25a56024ad53c8d965d4f8cfa4c..edf9ad06f8ad3c6b4f5b4955406c9128e73e105b 100644
--- a/chrome/browser/ui/cocoa/extensions/media_galleries_dialog_cocoa.mm
+++ b/chrome/browser/ui/cocoa/extensions/media_galleries_dialog_cocoa.mm
@@ -93,7 +93,9 @@ MediaGalleriesDialogCocoa::MediaGalleriesDialogCocoa(
controller_->HasPermittedGalleries()];
set_sheet(alert_);
- window_ = new ConstrainedWindowMac(controller->tab_contents(), this);
+ // May be NULL during tests.
+ if (controller->tab_contents())
+ window_ = new ConstrainedWindowMac(controller->tab_contents(), this);
}
MediaGalleriesDialogCocoa::~MediaGalleriesDialogCocoa() {
@@ -124,10 +126,12 @@ void MediaGalleriesDialogCocoa::SheetDidEnd(NSInteger result) {
switch (result) {
case NSAlertFirstButtonReturn:
accepted_ = true;
- window_->CloseConstrainedWindow();
+ if (window_)
+ window_->CloseConstrainedWindow();
break;
case NSAlertSecondButtonReturn:
- window_->CloseConstrainedWindow();
+ if (window_)
+ window_->CloseConstrainedWindow();
break;
default:
NOTREACHED();

Powered by Google App Engine
This is Rietveld 408576698