CookieStore: Remove reference counting.

content/public/browser/cookie_store_factory.h

 // Copyright (c) 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_COOKIE_STORE_FACTORY_H_
 #define CONTENT_PUBLIC_BROWSER_COOKIE_STORE_FACTORY_H_
 
 #include <string>
 #include <vector>
 
 #include "base/files/file_path.h"
-#include "base/memory/ref_counted.h"

[davidben, 2016/03/02 21:41:15]

I think you still need this include due to CookieStoreConfig fields.

[mmenke, 2016/03/02 21:46:29]

Thanks for catching that!

We need to make an automatic tool for IWYU issues.  The number of developer
hours wasted on that...

+#include "base/memory/scoped_ptr.h"
 #include "content/common/content_export.h"
 
 namespace base {
 class SequencedTaskRunner;
 }
 
 namespace net {
 class CookieCryptoDelegate;
 class CookieMonsterDelegate;
 class CookieStore;
@@ skipping 65 matching lines @@
   // |background_task_runner|.  If nullptr, uses a SequencedTaskRunner from the
   // BrowserThread blocking pool.
   //
   // Only used for persistent cookie stores.
   scoped_refptr<base::SequencedTaskRunner> background_task_runner;
 
   // If non-empty, overrides the default list of schemes that support cookies.
   std::vector<std::string> cookieable_schemes;
 };
 
-CONTENT_EXPORT net::CookieStore* CreateCookieStore(
+CONTENT_EXPORT scoped_ptr<net::CookieStore> CreateCookieStore(
     const CookieStoreConfig& config);
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_COOKIE_STORE_FACTORY_H_