Prevent inline CRX installs in popup windows.

chrome/browser/extensions/webstore_installer_test.h

Index: chrome/browser/extensions/webstore_installer_test.h
diff --git a/chrome/browser/extensions/webstore_installer_test.h b/chrome/browser/extensions/webstore_installer_test.h
index 9dff96423cdccc9e00917e4fb6df2af7f0614e19..0bf14a57a399483ed6bb0d27dd5eae37fcb0a39f 100644
--- a/chrome/browser/extensions/webstore_installer_test.h
+++ b/chrome/browser/extensions/webstore_installer_test.h
@@ -15,6 +15,10 @@ namespace base {
 class CommandLine;
 }  // namespace base
 
+namespace contents {
+class WebContents;
+}  // namespace contents

[Devlin, 2015/02/12 17:18:21]

nit: We usually don't have the "// namespace foo" for these really tiny forward
declarations.  Why don't we take this one out, and also the one on line 16.

[meacer, 2015/02/12 21:12:05]

Yeah, this looked odd to me too. Done.

+
 class WebstoreInstallerTest : public ExtensionBrowserTest {
  public:
   WebstoreInstallerTest(const std::string& webstore_domain,
@@ -34,6 +38,9 @@ class WebstoreInstallerTest : public ExtensionBrowserTest {
 
   void RunTest(const std::string& test_function_name);
 
+  void RunTest(content::WebContents* web_contents,
+               const std::string& test_function_name);
+
   // Passes |i| to |test_function_name|, and expects that function to
   // return one of "FAILED", "KEEPGOING" or "DONE". KEEPGOING should be
   // returned if more tests remain to be run and the current test succeeded,