CookieStore: Remove reference counting.

chrome/browser/profiles/profile_io_data.cc

Index: chrome/browser/profiles/profile_io_data.cc
diff --git a/chrome/browser/profiles/profile_io_data.cc b/chrome/browser/profiles/profile_io_data.cc
index 9e86ca9adb1a8e8c9ac1ea37b4657a8e031ea9d5..ef3a6be4cbaf10092c6bbe7e726e74a5bcb70f53 100644
--- a/chrome/browser/profiles/profile_io_data.cc
+++ b/chrome/browser/profiles/profile_io_data.cc
@@ -571,9 +571,9 @@ ProfileIOData::AppRequestContext::AppRequestContext() {
 }
 
 void ProfileIOData::AppRequestContext::SetCookieStore(
-    net::CookieStore* cookie_store) {
-  cookie_store_ = cookie_store;
-  set_cookie_store(cookie_store);
+    scoped_ptr<net::CookieStore> cookie_store) {
+  cookie_store_ = std::move(cookie_store);
+  set_cookie_store(cookie_store_.get());

[Randy Smith (Not in Mondays), 2016/02/29 22:37:45]

Not part of this CL, but I think it's dangerously confusing that the base
URLRequestContext has "pointer but no ownership" semantics for the things in it,
and this derived class has ownership semantics.   Dunno whether that's on your
radar screen (or should be), but I'll mention my aversive reaction in case it
has any data in it.

[mmenke, 2016/02/29 23:00:29]

I'd really like to move to a world where all URLRequestContexts have ownership
semantics.  Sharing guts has gotten us into all sorts of trouble.  Sharing stuff
is actually the exception, rather than the rule.  The main request context
doesn't own its stuff, but all other request contexts (Except those that share
pars of the main one) own everything.  Suppose the system and proxy request
contexts also share components.

In this particular case, there's a bug on file about giving apps their own
separate URLRequestContexts, since sharing allows certain sidechannel attacks
(Don't know the specifics, beyond things like DNS timing attacks).

 }
 
 void ProfileIOData::AppRequestContext::SetHttpTransactionFactory(