CookieStore: Remove reference counting.

headless/lib/browser/headless_url_request_context_getter.cc

 // Copyright 2016 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 "headless/lib/browser/headless_url_request_context_getter.h"
 
 #include "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/single_thread_task_runner.h"
 #include "base/threading/worker_pool.h"
 #include "content/public/browser/browser_thread.h"
 #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"
 #include "net/http/http_cache.h"
 #include "net/http/http_network_session.h"
 #include "net/http/http_server_properties_impl.h"
 #include "net/http/transport_security_state.h"
 #include "net/proxy/proxy_service.h"
 #include "net/ssl/channel_id_service.h"
 #include "net/ssl/default_channel_id_store.h"
@@ skipping 187 matching lines @@
 HeadlessURLRequestContextGetter::GetNetworkTaskRunner() const {
   return content::BrowserThread::GetMessageLoopProxyForThread(
       content::BrowserThread::IO);
 }
 
 net::HostResolver* HeadlessURLRequestContextGetter::host_resolver() const {
   return url_request_context_->host_resolver();
 }
 
 }  // namespace headless