Correctly integrate StoragePartition into TestingProfile.

components/autofill/content/browser/wallet/wallet_signin_helper_unittest.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 "components/autofill/content/browser/wallet/wallet_signin_helper.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.h"
@@ skipping 63 matching lines @@
   using WalletSigninHelper::state;
 
   // Bring in the State enum.
   using WalletSigninHelper::State;
   using WalletSigninHelper::IDLE;
 };
 
 }  // namespace
 
 class WalletSigninHelperTest : public testing::Test {
- public:
-  WalletSigninHelperTest() {}
-
+ protected:
   virtual void SetUp() OVERRIDE {
     signin_helper_.reset(new WalletSigninHelperForTesting(
         &mock_delegate_,
         browser_context_.GetRequestContext()));
     EXPECT_EQ(WalletSigninHelperForTesting::IDLE, state());
   }
 
   virtual void TearDown() OVERRIDE {
     signin_helper_.reset();
   }
 
- protected:
   // Sets up a response for the mock URLFetcher and completes the request.
   void SetUpFetcherResponseAndCompleteRequest(
       const std::string& url,
       int response_code,
       const net::ResponseCookies& cookies,
       const std::string& response_string) {
     net::TestURLFetcher* fetcher = factory_.GetFetcherByID(0);
     ASSERT_TRUE(fetcher);
     ASSERT_TRUE(fetcher->delegate());
 
@@ skipping 75 matching lines @@
 
   WalletSigninHelperForTesting::State state() const {
     return signin_helper_->state();
   }
 
   scoped_ptr<WalletSigninHelperForTesting> signin_helper_;
   MockWalletSigninHelperDelegate mock_delegate_;
   content::TestBrowserContext browser_context_;
 
  private:
+  // The profile's request context must be released on the IO thread.
+  content::TestBrowserThreadBundle thread_bundle_;
   net::TestURLFetcherFactory factory_;
-  content::TestBrowserThreadBundle thread_bundle_;
 };
 
 TEST_F(WalletSigninHelperTest, PassiveSigninSuccessful) {
   EXPECT_CALL(mock_delegate_, OnPassiveSigninSuccess("user@gmail.com"));
   signin_helper_->StartPassiveSignin();
   MockSuccessfulPassiveSignInResponse();
   MockSuccessfulGetAccountInfoResponse("user@gmail.com");
 }
 
 TEST_F(WalletSigninHelperTest, PassiveSigninFailedSignin404) {
@@ skipping 101 matching lines @@
   browser_context_.GetRequestContext()->GetURLRequestContext()
       ->set_cookie_store(cookie_monster);
   signin_helper_->StartWalletCookieValueFetch();
   base::RunLoop().RunUntilIdle();
 }
 
 // TODO(aruslan): http://crbug.com/188317 Need more tests.
 
 }  // namespace wallet
 }  // namespace autofill