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 when the UnauthorizedPluginInfoBar Finch trial is set to
+// Default behavior (disabled) should still be correctly tested when Finch trial
+// is over.
+// TODO(cthomp) Remove/simplify when Finch trial is completed
+class UnauthorizedPluginInfoBarBrowserTestBase : public InProcessBrowserTest {
+ public:
+  explicit UnauthorizedPluginInfoBarBrowserTestBase(
+      bool do_infobar_enabled_trial)
+    : do_infobar_enabled_trial_(do_infobar_enabled_trial) {}
+
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+    if (do_infobar_enabled_trial_) {
+      command_line->AppendSwitchASCII(switches::kForceFieldTrials,
+                                      "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();
+  }
+
+  // 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 {
+      return true;
+    }
+  }
+
+  bool InfoBarCountTest(unsigned int number_infobars_expected,
+                        Browser* browser) {
+    ServeContentTestData();
+    content::WebContents* contents =
+        browser->tab_strip_model()->GetActiveWebContents();
+    // ASSERT_TRUE(contents);

[radhikabhar, 2014/06/06 17:05:40]

Do we need this comment out here?

[Chris Thompson, 2014/06/06 17:37:48]

Uncommented and changed to EXPECT_TRUE (since these are mostly useful for
debugging a failed test, and the actual assertion we want to make is on whether
the infobar was shown or not).

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

FTR, ASSERT_ and EXPECT_ are not used to distinguish between the actual
assertion of the test and something else. ASSERT_ is for "fatal test failures",
which mean that the test can't possibly continue, so it will return from the
current method (which is why you can only use it in void methods). EXPECT_ is
for regular test failures from which you can recover.

+    InfoBarService* infobar_service = InfoBarService::FromWebContents(contents);
+    // ASSERT_TRUE(infobar_service);
+    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);
+    // Compare to number of infobars expected
+    return (number_infobars_expected == infobar_service->infobar_count());
+  }
+
+ private:
+  bool do_infobar_enabled_trial_;
+  DISALLOW_COPY_AND_ASSIGN(UnauthorizedPluginInfoBarBrowserTestBase);
+};
+
+class InfoBarEnabledBrowserTest :
+  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;
+  }
+  ASSERT_TRUE(InfoBarCountTest(1u, browser()));
+}
+
+// When "UnauthorizedPluginInfoBar" is "Disabled", infobar for NPAPI plugin
+// should not be shown.
+IN_PROC_BROWSER_TEST_F(InfoBarDisabledBrowserTest, TestDisabled) {
+  if (!TestPageExists()) {
+    LOG(INFO) <<
+        "Test skipped because plugin/quicktime.html test file wasn't found.";
+    return;
+  }
+  ASSERT_TRUE(InfoBarCountTest(0u, browser()));
+}
+
+} // namespace