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//extensions/browser/api/feedback_private.

This CL was uploaded by git cl split.

R=afakhry@chromium.org

Bug: 809610
Change-Id: I41eb7e5f4a7b0286c0e3338e1b1792f38c7fdfdb
Reviewed-on: https://chromium-review.googlesource.com/c/1259017
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596593}

extensions/browser/api/feedback_private/feedback_private_api_unittest_base_chromeos.cc

@@ -136,8 +136,9 @@ void FeedbackPrivateApiUnittestBase::SetUp() {
   // that there is always a valid ApiResourceManager<LogSourceResource> when
   // subsequent unit tests are running.
   ApiResourceManager<LogSourceResource>::GetFactoryInstance()
-      ->SetTestingFactoryAndUse(browser_context(),
-                                ApiResourceManagerTestFactory);
+      ->SetTestingFactoryAndUse(
+          browser_context(),
+          base::BindRepeating(&ApiResourceManagerTestFactory));
 }
 
 void FeedbackPrivateApiUnittestBase::TearDown() {