[Extensions] Wait until the Window object is cleared before classifying contexts

chrome/browser/extensions/extension_bindings_apitest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Contains holistic tests of the bindings infrastructure
 
 #include "chrome/browser/extensions/api/permissions/permissions_api.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/net/url_request_mock_util.h"
 #include "chrome/browser/ui/browser.h"
@@ skipping 139 matching lines @@
   // See handler_function_type_checking.html.
   std::string result;
   EXPECT_TRUE(content::ExecuteScriptAndExtractString(
       web_contents,
       "window.domAutomationController.send("
           "document.getElementById('status').textContent.trim());",
       &result));
   EXPECT_EQ("success", result);
 }
 
+// Regression test for crbug.com/340382, which was having extension bindings
+// flakily undefined because the script context was improperly classified.
+// Thanks to jhawkins@ for the only known reliable repro.
+IN_PROC_BROWSER_TEST_F(ExtensionBindingsApiTest,
+                       RegressionTestForUndefinedBindings) {
+  // This is important - likely because the signin page does crazy stuff. This
+  // doesn't repro if we don't navigate to the signin page.
+  ui_test_utils::NavigateToURL(browser(), GURL("chrome://chrome-signin"));
+
+  ASSERT_TRUE(RunPlatformAppTest("bindings/chromei18nrepro")) << message_;
+}
+
 }  // namespace
 }  // namespace extensions