Convert indirect uses of TestingFactoryFunction

TestingFactoryFunction is a simple function pointer. It is
deprecated in favor of TestingFactory which is a Callback<>.
Convert indirect uses by using base::BindRepeating() in all
invocation of SetTestingFactory/AddTestingFactory.

This converts uses in src//chrome/browser/safe_browsing.

This CL was uploaded by git cl split.

R=nparker@chromium.org

Bug: 809610
Change-Id: I880627bdc67561e465ef16eae0bf1d3ad66c234d
Reviewed-on: https://chromium-review.googlesource.com/c/1297415Reviewed-by: Nathan Parker <nparker@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602385}

chrome/browser/safe_browsing/download_protection/download_protection_service_unittest.cc

@@ -266,7 +266,9 @@ class DownloadProtectionServiceTest : public ChromeRenderViewHostTestHarness {
     // |test_event_router_| is owned by KeyedServiceFactory.
     test_event_router_ = extensions::CreateAndUseTestEventRouter(profile());
     extensions::SafeBrowsingPrivateEventRouterFactory::GetInstance()
-        ->SetTestingFactory(profile(), BuildSafeBrowsingPrivateEventRouter);
+        ->SetTestingFactory(
+            profile(),
+            base::BindRepeating(&BuildSafeBrowsingPrivateEventRouter));
   }
 
   void TearDown() override {