CookieStore: Remove reference counting.

headless/lib/browser/headless_url_request_context_getter.cc

Index: headless/lib/browser/headless_url_request_context_getter.cc
diff --git a/headless/lib/browser/headless_url_request_context_getter.cc b/headless/lib/browser/headless_url_request_context_getter.cc
index 4e8a195458811ad8e85d5652e8a59fdd781aebdf..de1ad37fcd29dd2d762c1103973b410d70c1adfa 100644
--- a/headless/lib/browser/headless_url_request_context_getter.cc
+++ b/headless/lib/browser/headless_url_request_context_getter.cc
@@ -12,6 +12,7 @@
 #include "content/public/browser/cookie_store_factory.h"
 #include "content/public/common/content_switches.h"
 #include "net/cert/cert_verifier.h"
+#include "net/cookies/cookie_store.h"

[Randy Smith (Not in Mondays), 2016/03/01 20:48:43]

Hmmm.  As best I can tell, this type is only used implicitly, in the argument to
one function and the return value from another.  I'd think that both of those
situations would be better handled by the include files that declare those
functions, so my inclination would be that they wouldn't provoke an IWYU
inclusion.  But I may be misunderstanding how IWYU should be implemented.

[mmenke, 2016/03/01 22:44:45]

Both of those headers forward declare the class and only use it in scoped_ptrs,
but this file needs access to its destructor, since if (briefly) holds on to a
scoped_ptr.  Including those files isn't enough, unfortunately.  Get a build
error otherwise.

 #include "net/dns/host_resolver.h"
 #include "net/dns/mapped_host_resolver.h"
 #include "net/http/http_auth_handler_factory.h"