Remove the unused close_on_interstitial_webui setting.

components/web_modal/web_contents_modal_dialog_manager_unittest.cc

Index: components/web_modal/web_contents_modal_dialog_manager_unittest.cc
diff --git a/components/web_modal/web_contents_modal_dialog_manager_unittest.cc b/components/web_modal/web_contents_modal_dialog_manager_unittest.cc
index b73d617b56af00ba3f68506ac4f52f808cd190e3..aff688d4c003ad643bd01acef21a930f2c9c4735 100644
--- a/components/web_modal/web_contents_modal_dialog_manager_unittest.cc
+++ b/components/web_modal/web_contents_modal_dialog_manager_unittest.cc
@@ -241,45 +241,34 @@ TEST_F(WebContentsModalDialogManagerTest, VisibilityObservation) {
   native_manager->StopTracking();
 }
 
-// Test that attaching an interstitial page closes dialogs configured to close.
+// Test that attaching an interstitial page closes all dialogs.
 TEST_F(WebContentsModalDialogManagerTest, InterstitialPage) {
   const NativeWebContentsModalDialog dialog1 = MakeFakeDialog();
   const NativeWebContentsModalDialog dialog2 = MakeFakeDialog();
-  const NativeWebContentsModalDialog dialog3 = MakeFakeDialog();
 
   NativeManagerTracker tracker1;
   NativeManagerTracker tracker2;
-  NativeManagerTracker tracker3;
   TestNativeWebContentsModalDialogManager* native_manager1 =
       new TestNativeWebContentsModalDialogManager(dialog1, manager, &tracker1);
   TestNativeWebContentsModalDialogManager* native_manager2 =
       new TestNativeWebContentsModalDialogManager(dialog2, manager, &tracker2);
-  TestNativeWebContentsModalDialogManager* native_manager3 =
-      new TestNativeWebContentsModalDialogManager(dialog3, manager, &tracker3);
   manager->ShowDialogWithManager(dialog1,
       scoped_ptr<SingleWebContentsDialogManager>(native_manager1).Pass());
   manager->ShowDialogWithManager(dialog2,
       scoped_ptr<SingleWebContentsDialogManager>(native_manager2).Pass());
-  manager->ShowDialogWithManager(dialog3,
-      scoped_ptr<SingleWebContentsDialogManager>(native_manager3).Pass());
-
-#if defined(OS_WIN) || defined(USE_AURA)
-  manager->SetCloseOnInterstitialPage(dialog2, false);
-#else
-  // TODO(wittman): Remove this section once Mac is changed to close on
-  // interstitial pages by default.
-  manager->SetCloseOnInterstitialPage(dialog1, true);
-  manager->SetCloseOnInterstitialPage(dialog3, true);
-#endif
 
   test_api->DidAttachInterstitialPage();
 
+#if defined(USE_AURA)
   EXPECT_EQ(NativeManagerTracker::CLOSED, tracker1.state_);
-  EXPECT_EQ(NativeManagerTracker::SHOWN, tracker2.state_);
-  EXPECT_EQ(NativeManagerTracker::CLOSED, tracker3.state_);
+  EXPECT_EQ(NativeManagerTracker::CLOSED, tracker2.state_);
+#else
+  EXPECT_EQ(NativeManagerTracker::SHOWN, tracker1.state_);
+  EXPECT_EQ(NativeManagerTracker::NOT_SHOWN, tracker2.state_);
+#endif
+
   EXPECT_TRUE(tracker1.was_shown_);
-  EXPECT_TRUE(tracker2.was_shown_);
-  EXPECT_FALSE(tracker3.was_shown_);
+  EXPECT_FALSE(tracker2.was_shown_);
 
   native_manager2->StopTracking();

[Mike Wittman, 2014/05/16 21:37:32]

This should be something like:

#if !defined(USE_AURA)
  native_manager1->StopTracking();
  native_manager2->StopTracking();
#endif

The native managers are deleted when the windows are closed in the USE_AURA
case.

[msw, 2014/05/16 22:38:29]

Done.

 }