Removed NPAPI plugin infobar, with Finch trial gate (disabled by default).

chrome/renderer/chrome_content_renderer_client_browsertest.cc

 // Copyright 2013 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.
 
 #include "chrome/renderer/chrome_content_renderer_client.h"
 
 #include <vector>
 
+#include "base/bind.h"
 #include "base/command_line.h"
+#include "base/files/file_path.h"
+#include "base/path_service.h"
+#include "chrome/browser/infobars/infobar_service.h"
+#include "chrome/browser/net/url_request_mock_util.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/renderer/chrome_content_renderer_client.h"
 #include "chrome/test/base/chrome_render_view_test.h"
+#include "chrome/test/base/in_process_browser_test.h"
+#include "chrome/test/base/ui_test_utils.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/common/content_paths.h"
+#include "content/public/common/content_switches.h"
+#include "content/public/test/browser_test_utils.h"
+#include "content/public/test/test_utils.h"
+#include "content/test/net/url_request_mock_http_job.h"
 #include "third_party/WebKit/public/web/WebLocalFrame.h"
 #include "url/gurl.h"
 
+using content::BrowserThread;
+using content::URLRequestMockHTTPJob;
+
 typedef ChromeRenderViewTest InstantProcessNavigationTest;
 
 // Tests that renderer-initiated navigations from an Instant render process get
 // bounced back to the browser to be rebucketed into a non-Instant renderer if
 // necessary.
 TEST_F(InstantProcessNavigationTest, ForkForNavigationsFromInstantProcess) {
   CommandLine::ForCurrentProcess()->AppendSwitch(switches::kInstantProcess);
   bool unused;
   ChromeContentRendererClient* client =
       static_cast<ChromeContentRendererClient*>(content_renderer_client_.get());
@@ skipping 18 matching lines @@
   EXPECT_TRUE(client->ShouldFork(
       GetMainFrame(), GURL("http://example.com/newtab"), "GET", false, false,
       &unused));
   EXPECT_TRUE(client->ShouldFork(
       GetMainFrame(), GURL("http://example.com/search?q=foo"), "GET", false,
       false, &unused));
   EXPECT_FALSE(client->ShouldFork(
       GetMainFrame(), GURL("http://example.com/"), "GET", false, false,
       &unused));
 }
+
+namespace {
+
+// Tests that the NPAPI unauthorized plugin infobar is:
+// (1) Shown when the Finch trial is set to "Enabled"
+// (2) Not shown when the Finch trial is set to "Disabled"
+// TODO(cthomp) Remove/simplify when Finch trial is completed crbug.com/381944

[Bernhard Bauer, 2014/06/09 17:03:18]

Nit: The recommended format for TODOs is TODO(ldap): (colon after the closing
parenthesis).

+class UnauthorizedPluginInfoBarBrowserTestBase : public InProcessBrowserTest {
+ public:
+  explicit UnauthorizedPluginInfoBarBrowserTestBase(
+      bool infobar_trial_enabled)
+    : infobar_trial_enabled_(infobar_trial_enabled) {}

[Bernhard Bauer, 2014/06/09 17:03:18]

Nit: indent two more spaces.

+
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+    if (infobar_trial_enabled_) {
+      command_line->AppendSwitchASCII(switches::kForceFieldTrials,

[Bernhard Bauer, 2014/06/09 17:03:18]

You can enable field trials directly with FieldTrialList::CreateTrialsFromString
(and the same string argument as on the command line), then you can enable them
whenever you want.

+                                      "UnauthorizedPluginInfoBar/Enabled/");
+    } else {
+      command_line->AppendSwitchASCII(switches::kForceFieldTrials,
+                                      "UnauthorizedPluginInfoBar/Disabled/");
+    }
+  }
+
+  // Makes URLRequestMockHTTPJobs serve data from content::DIR_TEST_DATA
+  // instead of chrome::DIR_TEST_DATA.
+  void ServeContentTestData() {
+    base::FilePath root_http;
+    PathService::Get(content::DIR_TEST_DATA, &root_http);
+    BrowserThread::PostTaskAndReply(
+        BrowserThread::IO, FROM_HERE,
+        base::Bind(URLRequestMockHTTPJob::AddUrlHandler, root_http),
+        base::MessageLoop::current()->QuitWhenIdleClosure());
+    content::RunMessageLoop();

[Bernhard Bauer, 2014/06/09 17:03:18]

Prefer a MessageLoopRunner with an explicit quit closure that you can pass to
PostTaskAndReply above.

+  }
+
+  // Verifies that the test page exists (only present with src-internal)
+  bool TestPageExists() {
+    if (!base::PathExists(ui_test_utils::GetTestFilePath(
+        base::FilePath(FILE_PATH_LITERAL("plugin")),
+        base::FilePath(FILE_PATH_LITERAL("quicktime.html"))))) {
+      return false;
+    } else {

[Bernhard Bauer, 2014/06/09 17:03:18]

`else` isn't necessary if the `if` branch returns.

You could also simplify this to to just return base::PathExists(...);

+      return true;
+    }
+  }
+
+  void InfoBarCountTest(unsigned int number_infobars_expected,

[Bernhard Bauer, 2014/06/09 17:03:18]

Don't use `unsigned int`, use a regular int or a specific type. In this case,
you want to compare the size of a container, so use size_t.

+                        Browser* browser) {
+    ServeContentTestData();
+    content::WebContents* contents =
+        browser->tab_strip_model()->GetActiveWebContents();
+    EXPECT_TRUE(contents != NULL);

[Bernhard Bauer, 2014/06/09 17:03:18]

If contents is actually NULL, you will crash below. If you want to handle that
case graceful, you can ASSERT. Also, the comparison to NULL is unnecessary.

+    InfoBarService* infobar_service = InfoBarService::FromWebContents(contents);
+    EXPECT_TRUE(infobar_service != NULL);
+    EXPECT_EQ(0u, infobar_service->infobar_count());
+
+    base::FilePath path(FILE_PATH_LITERAL("plugin/quicktime.html"));
+    GURL url(URLRequestMockHTTPJob::GetMockUrl(path));
+    ui_test_utils::NavigateToURL(browser, url);
+    ASSERT_EQ(number_infobars_expected, infobar_service->infobar_count());
+  }
+
+ private:
+  bool infobar_trial_enabled_;
+  DISALLOW_COPY_AND_ASSIGN(UnauthorizedPluginInfoBarBrowserTestBase);
+};
+
+class InfoBarEnabledBrowserTest :

[Bernhard Bauer, 2014/06/09 17:03:18]

Do you actually need these two test fixtures? Just set |infobar_trial_enabled_|
when you need to.

+  public UnauthorizedPluginInfoBarBrowserTestBase {
+ public:
+  InfoBarEnabledBrowserTest() :
+    UnauthorizedPluginInfoBarBrowserTestBase(true) {}
+};
+
+class InfoBarDisabledBrowserTest :
+  public UnauthorizedPluginInfoBarBrowserTestBase {
+ public:
+  InfoBarDisabledBrowserTest() :
+    UnauthorizedPluginInfoBarBrowserTestBase(false) {}
+};
+
+// When "UnauthorizedPluginInfoBar" is "Enabled", infobar for NPAPI plugin
+// should be shown.
+IN_PROC_BROWSER_TEST_F(InfoBarEnabledBrowserTest, InfobarEnabled) {
+  if (!TestPageExists()) {
+    LOG(INFO) <<
+        "Test skipped because plugin/quicktime.html test file wasn't found.";
+    return;
+  }
+  InfoBarCountTest(1u, browser());
+}
+
+// When "UnauthorizedPluginInfoBar" is "Disabled", infobar for NPAPI plugin
+// should not be shown.
+IN_PROC_BROWSER_TEST_F(InfoBarDisabledBrowserTest, InfobarDisabled) {
+  if (!TestPageExists()) {
+    LOG(INFO) <<
+        "Test skipped because plugin/quicktime.html test file wasn't found.";
+    return;
+  }
+  InfoBarCountTest(0u, browser());
+}
+
+} // namespace