Permission Action Reporting: Make the browsertest work with SyncTest framework.

chrome/browser/safe_browsing/permission_reporter_browsertest.cc

 // Copyright 2016 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/browser/safe_browsing/permission_reporter.h"
 
 #include "base/command_line.h"
 #include "base/memory/ptr_util.h"
 #include "base/memory/ref_counted.h"
+#include "base/run_loop.h"
 #include "base/test/simple_test_clock.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/permissions/permission_request_manager.h"
+#include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/safe_browsing/mock_permission_report_sender.h"
 #include "chrome/browser/safe_browsing/ping_manager.h"
 #include "chrome/browser/safe_browsing/safe_browsing_service.h"
+#include "chrome/browser/sync/test/integration/sync_test.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/browser/ui/website_settings/mock_permission_prompt_factory.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/safe_browsing/permission_report.pb.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"
 
 #if !defined(OS_CHROMEOS)
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "components/signin/core/browser/signin_manager.h"
 #endif
 
 namespace safe_browsing {
 
-class PermissionReporterBrowserTest : public InProcessBrowserTest {
+class PermissionReporterBrowserTest : public SyncTest {
+ public:
+  PermissionReporterBrowserTest() : SyncTest(SINGLE_CLIENT) {}
+  ~PermissionReporterBrowserTest() override {}
+
  protected:
   void SetUpOnMainThread() override {
-    InProcessBrowserTest::SetUpOnMainThread();
+    SyncTest::SetUpOnMainThread();
 
     base::RunLoop run_loop;
     content::BrowserThread::PostTaskAndReply(
         content::BrowserThread::IO, FROM_HERE,
         base::Bind(
             &PermissionReporterBrowserTest::AttachMockReportSenderOnIOThread,
             base::Unretained(this),
             make_scoped_refptr(g_browser_process->safe_browsing_service())),
         run_loop.QuitClosure());
     run_loop.Run();
-
-    PermissionRequestManager* manager = GetPermissionRequestManager();
-    mock_permission_prompt_factory_ =
-        base::MakeUnique<MockPermissionPromptFactory>(manager);
-    manager->DisplayPendingRequests();
-
-#if !defined(OS_CHROMEOS)
-    SigninManager* signin_manager =
-        SigninManagerFactory::GetForProfile(browser()->profile());
-    signin_manager->SetAuthenticatedAccountInfo("gaia_id", "fake_username");
-#endif
-  }
-
-  void TearDownOnMainThread() override {
-    InProcessBrowserTest::TearDownOnMainThread();
-    mock_permission_prompt_factory_.reset();
   }
 
   void SetUpCommandLine(base::CommandLine* command_line) override {
+    SyncTest::SetUpCommandLine(command_line);
     command_line->AppendSwitch(switches::kEnablePermissionActionReporting);
   }
 
   void AttachMockReportSenderOnIOThread(
       scoped_refptr<SafeBrowsingService> safe_browsing_service) {
     DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
 
     mock_report_sender_ = new MockPermissionReportSender;
 
     safe_browsing_service->ping_manager()->permission_reporter_.reset(
         new PermissionReporter(base::WrapUnique(mock_report_sender_),
                                base::WrapUnique(new base::SimpleTestClock)));
   }
 
-  PermissionRequestManager* GetPermissionRequestManager() {
+  PermissionRequestManager* GetPermissionRequestManager(Browser* browser) {
     return PermissionRequestManager::FromWebContents(
-        browser()->tab_strip_model()->GetActiveWebContents());
+        browser->tab_strip_model()->GetActiveWebContents());
   }
 
-  void AcceptBubble() { GetPermissionRequestManager()->Accept(); }
-
-  MockPermissionPromptFactory* prompt_factory() {
-    return mock_permission_prompt_factory_.get();
-  }
+  void AcceptBubble(Browser* b) { GetPermissionRequestManager(b)->Accept(); }
 
   MockPermissionReportSender* mock_report_sender() {
     return mock_report_sender_;
   }
 
  private:
-  std::unique_ptr<MockPermissionPromptFactory> mock_permission_prompt_factory_;
-
   // Owned by permission reporter.
   MockPermissionReportSender* mock_report_sender_;
+
+  DISALLOW_COPY_AND_ASSIGN(PermissionReporterBrowserTest);
 };
 
 // Test that permission action report will be sent if the user is opted into it.
-// TODO(kcarattini): Make this test pass with the call to Sync backends.
 // TODO(kcarattini): Address crbug/638316 to reenable this test.
 IN_PROC_BROWSER_TEST_F(PermissionReporterBrowserTest,
                        DISABLED_PermissionActionReporting) {
+  // Set up the Sync client.
+  ASSERT_TRUE(SetupSync());
+  Profile* profile = GetProfile(0);
+  Browser* browser = CreateBrowser(profile);
+
+  // Set up mock permission manager and prompt factory.
+  PermissionRequestManager* manager = GetPermissionRequestManager(browser);

[Nathan Parker, 2016/08/24 05:19:19]

I'm not familiar with where this should be done (here or in SetUp, or
SetUpOnMainThread), but if this works, LGTM.

[kcarattini, 2016/08/24 06:00:12]

The reason I moved this was that the fake profile that is set up in SyncTest
doesn't happen until the SetupSync call, and I need all this to use the browser
object that is created with that fake profile. Maybe it's not the cleanest, but
it works :).

+  std::unique_ptr<MockPermissionPromptFactory> mock_permission_prompt_factory =
+      base::MakeUnique<MockPermissionPromptFactory>(manager);
+  manager->DisplayPendingRequests();
+
   ASSERT_TRUE(embedded_test_server()->Start());
-
   ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(
-      browser(), embedded_test_server()->GetURL("/permissions/request.html"),
+      browser, embedded_test_server()->GetURL("/permissions/request.html"),
       1);
 
-  prompt_factory()->WaitForPermissionBubble();
-  EXPECT_TRUE(prompt_factory()->is_visible());
+  mock_permission_prompt_factory->WaitForPermissionBubble();
+  EXPECT_TRUE(mock_permission_prompt_factory->is_visible());
 
-  AcceptBubble();
+  AcceptBubble(browser);
 
-  EXPECT_FALSE(prompt_factory()->is_visible());
+  EXPECT_FALSE(mock_permission_prompt_factory->is_visible());
   EXPECT_EQ(1, mock_report_sender()->GetAndResetNumberOfReportsSent());
 
   PermissionReport permission_report;
   ASSERT_TRUE(
       permission_report.ParseFromString(mock_report_sender()->latest_report()));
   EXPECT_EQ(PermissionReport::GEOLOCATION, permission_report.permission());
   EXPECT_EQ(PermissionReport::GRANTED, permission_report.action());
   EXPECT_EQ(embedded_test_server()->base_url().spec(),
             permission_report.origin());
   EXPECT_EQ(PermissionReport::DESKTOP_PLATFORM,
             permission_report.platform_type());
   EXPECT_EQ(PermissionReport::NO_GESTURE, permission_report.gesture());
   EXPECT_EQ(PermissionReport::PERSIST_DECISION_UNSPECIFIED,
             permission_report.persisted());
   EXPECT_EQ(0, permission_report.num_prior_dismissals());
   EXPECT_EQ(0, permission_report.num_prior_ignores());
 }
 
 }  // namespace safe_browsing