Stop using nested message loop for alert() and other JS dialogs

chrome/browser/extensions/content_script_apitest.cc

Index: chrome/browser/extensions/content_script_apitest.cc
diff --git a/chrome/browser/extensions/content_script_apitest.cc b/chrome/browser/extensions/content_script_apitest.cc
index b13f94306b37cedbbb469634143639137cd32f71..c8d442885dff659576a6f2f6fc95385d298d0f84 100644
--- a/chrome/browser/extensions/content_script_apitest.cc
+++ b/chrome/browser/extensions/content_script_apitest.cc
@@ -482,7 +482,6 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, ContentScriptBlockingScript) {
   run_loop.Run();
   // Right now, the alert dialog is showing and blocking injection of anything
   // after it, so the listener shouldn't be satisfied.
-  EXPECT_TRUE(RunAllPending(web_contents));
   EXPECT_FALSE(listener.was_satisfied());
   EXPECT_EQ(1u, dialog_helper.dialog_count());
   dialog_helper.CloseDialogs();
@@ -538,7 +537,6 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, ContentScriptBlockingScriptTabClosed) {
   // Now, instead of closing the dialog, just close the tab. Later scripts
   // should never get a chance to run (and we shouldn't crash).
   run_loop.Run();
-  EXPECT_TRUE(RunAllPending(web_contents));
   EXPECT_FALSE(listener.was_satisfied());
   EXPECT_TRUE(browser()->tab_strip_model()->CloseWebContentsAt(
       browser()->tab_strip_model()->active_index(), 0));
@@ -576,7 +574,6 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest,
   run_loop.Run();
 
   // The extension will have injected at idle, but it should only inject once.
-  EXPECT_TRUE(RunAllPending(web_contents));
   EXPECT_EQ(1u, dialog_helper.dialog_count());
   dialog_helper.CloseDialogs();
   EXPECT_TRUE(RunAllPending(web_contents));