Remove TestSigninClient's reference to SharedURLLoaderFactory instance

We can simply call TestURLLoaderFactory::GetSafeWeakWrapper as needed.

BUG=None

Change-Id: Id11b4942efc4b786433ae7def80f2c6374352c61
Reviewed-on: https://chromium-review.googlesource.com/c/1468204
Reviewed-by: David Roger <droger@chromium.org>
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#631757}

components/signin/core/browser/test_signin_client.cc

@@ -13,10 +13,7 @@
 #include "testing/gtest/include/gtest/gtest.h"
 
 TestSigninClient::TestSigninClient(PrefService* pref_service)
-    : shared_factory_(
-          base::MakeRefCounted<network::WeakWrapperSharedURLLoaderFactory>(
-              &test_url_loader_factory_)),
-      pref_service_(pref_service),
+    : pref_service_(pref_service),
       are_signin_cookies_allowed_(true),
       network_calls_delayed_(false),
       is_signout_allowed_(true) {}
@@ -39,7 +36,7 @@ void TestSigninClient::PreSignOut(
 
 scoped_refptr<network::SharedURLLoaderFactory>
 TestSigninClient::GetURLLoaderFactory() {
-  return shared_factory_;
+  return test_url_loader_factory_.GetSafeWeakWrapper();
 }
 
 network::mojom::CookieManager* TestSigninClient::GetCookieManager() {

components/signin/core/browser/test_signin_client.h

@@ -91,7 +91,6 @@ class TestSigninClient : public SigninClient {
 
  private:
   network::TestURLLoaderFactory test_url_loader_factory_;
-  scoped_refptr<network::SharedURLLoaderFactory> shared_factory_;
 
   PrefService* pref_service_;
   std::unique_ptr<network::mojom::CookieManager> cookie_manager_;