Correctly integrate StoragePartition into TestingProfile.

components/autofill/core/browser/autofill_manager_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 <algorithm>
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 722 matching lines @@
   std::vector<int> autofill_unique_ids_;
 
   DISALLOW_COPY_AND_ASSIGN(TestAutofillExternalDelegate);
 };
 
 }  // namespace
 
 class AutofillManagerTest : public ChromeRenderViewHostTestHarness {
  public:
   virtual void SetUp() OVERRIDE {
     TestingProfile* profile = CreateProfile();

[tfarina, 2013/07/11 01:35:45]

can we create TestBrowserContext now?

[awong, 2013/07/11 21:04:15]

This ended up not being a super simple change so I'd rather leave it out of this
CL which is big enough as is.

Also, since AutofillManagerTest derives from ChromeRenderViewHostTestHarness,
using TestingProfile seems like it makes more sense...

-    profile->CreateRequestContext();
     browser_context_.reset(profile);
     autofill::PersonalDataManagerFactory::GetInstance()->SetTestingFactory(
         profile, TestPersonalDataManager::Build);
 
     ChromeRenderViewHostTestHarness::SetUp();
 
     autofill::TabAutofillManagerDelegate::CreateForWebContents(web_contents());
 
     personal_data_.SetBrowserContext(profile);
     autofill_driver_.reset(new MockAutofillDriver(web_contents()));
@@ skipping 2518 matching lines @@
   forms.push_back(user_supplied);
 
   // FormStructure should contain the same forms as before.
   DynamicFormsSeen(forms);
   form_structures = autofill_manager_->GetFormStructures();
   ASSERT_EQ(1U, form_structures.size());
   EXPECT_EQ("/form.html", form_structures[0]->source_url().path());
 }
 
 }  // namespace autofill