Remove the Extensions URLRequestContext

chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc

 // Copyright (c) 2012 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 "chrome/browser/browsing_data/browsing_data_cookie_helper.h"
 
 #include "base/bind.h"
 #include "base/run_loop.h"
 #include "chrome/test/base/testing_profile.h"
+#include "content/public/browser/storage_partition.h"
+#include "content/public/common/url_constants.h"
 #include "content/public/test/test_browser_thread_bundle.h"
+#include "content/public/test/test_utils.h"
 #include "net/cookies/canonical_cookie.h"
 #include "net/cookies/parsed_cookie.h"
 #include "net/url_request/url_request_context_getter.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
 class BrowsingDataCookieHelperTest : public testing::Test {
  public:
   BrowsingDataCookieHelperTest()
       : testing_profile_(new TestingProfile()) {
   }
 
+  net::CookieMonster* GetCookieMonster(const GURL& origin) {
+    using content::BrowserContext;

[Bernhard Bauer, 2013/08/16 06:46:44]

This isn't very... commonly seen in Chrome code. Any reason you're not just
doing the import at top level?

[awong, 2013/08/17 00:32:52]

Since no one else was using it, didn't see a reason to pull it up higher.  If
you'd like, I can pull it up to the top level. Don't really care too much.

+    // Since it's a unittest, assume default StoragePartition.
+    return content::GetCookieStoreForScheme(
+        BrowserContext::GetDefaultStoragePartition(testing_profile_.get()),
+        origin.scheme())->GetCookieMonster();
+  }
+
   void CreateCookiesForTest() {
-    scoped_refptr<net::CookieMonster> cookie_monster =
-        testing_profile_->GetCookieMonster();
-    cookie_monster->SetCookieWithOptionsAsync(
-        GURL("http://www.google.com"), "A=1", net::CookieOptions(),
+    GURL origin1("http://www.google.com");
+    GURL origin2("http://www.gmail.google.com");
+    GetCookieMonster(origin1)->SetCookieWithOptionsAsync(
+        origin1, "A=1", net::CookieOptions(),
         net::CookieMonster::SetCookiesCallback());
-    cookie_monster->SetCookieWithOptionsAsync(
-        GURL("http://www.gmail.google.com"), "B=1", net::CookieOptions(),
+    GetCookieMonster(origin2)->SetCookieWithOptionsAsync(
+        origin2, "B=1", net::CookieOptions(),
         net::CookieMonster::SetCookiesCallback());
   }
 
   void CreateCookiesForDomainCookieTest() {
-    scoped_refptr<net::CookieMonster> cookie_monster =
-        testing_profile_->GetCookieMonster();
+    GURL origin("http://www.google.com");
+    scoped_refptr<net::CookieMonster> cookie_monster = GetCookieMonster(origin);
     cookie_monster->SetCookieWithOptionsAsync(
-        GURL("http://www.google.com"), "A=1", net::CookieOptions(),
+        origin, "A=1", net::CookieOptions(),
         net::CookieMonster::SetCookiesCallback());
     cookie_monster->SetCookieWithOptionsAsync(
-        GURL("http://www.google.com"), "A=2; Domain=.www.google.com ",
+        origin, "A=2; Domain=.www.google.com ",
         net::CookieOptions(), net::CookieMonster::SetCookiesCallback());
   }
 
   void FetchCallback(const net::CookieList& cookies) {
     ASSERT_EQ(2UL, cookies.size());
     cookie_list_ = cookies;
     net::CookieList::const_iterator it = cookies.begin();
 
     // Correct because fetching cookies will get a sorted cookie list.
     ASSERT_TRUE(it != cookies.end());
@@ skipping 381 matching lines @@
   //   "A=1;
   //   "A=3; Domain=www.google.com"
   // Add a domain cookie and check if it increases the cookie count.
   helper->AddChangedCookie(frame2_url, frame1_url,
                            cookie_pair4 + "; Domain=" + cookie_domain,
                            net::CookieOptions());
   EXPECT_EQ(5U, helper->GetCookieCount());
 }
 
 }  // namespace