Instantiate OriginBoundCertService in relevant places and do plumbing to pass it down to HttpNetw...

chrome/browser/profiles/profile_io_data.h

 // Copyright (c) 2011 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 CHROME_BROWSER_PROFILES_PROFILE_IO_DATA_H_
 #define CHROME_BROWSER_PROFILES_PROFILE_IO_DATA_H_
 #pragma once
 
 #include <set>
 #include "base/basictypes.h"
@@ skipping 21 matching lines @@
 
 namespace fileapi {
 class FileSystemContext;
 }  // namespace fileapi
 
 namespace net {
 class CookieStore;
 class DnsCertProvenanceChecker;
 class HttpTransactionFactory;
 class NetLog;
+class OriginBoundCertService;
 class ProxyConfigService;
 class ProxyService;
 class SSLConfigService;
 class TransportSecurityState;
 }  // namespace net
 
 namespace prerender {
 class PrerenderManager;
 };  // namespace prerender
 
@@ skipping 148 matching lines @@
   void ShutdownOnUIThread();
 
   BooleanPrefMember* enable_referrers() const {
     return &enable_referrers_;
   }
 
   ChromeURLDataManagerBackend* chrome_url_data_manager_backend() const {
     return chrome_url_data_manager_backend_.get();
   }
 
+  void set_origin_bound_cert_service(

[wtc, 2011/08/09 17:43:45]

It is important to point out that this is not a typical setter.

Please document that OriginBoundCertService is created by
a derived class of ProfileIOData and the derived class calls
this method to set the origin_bound_cert_service_ member
and transfers ownership to the base class.

An alternative design is to make origin_bound_cert_service_
a protected member.  But I am not sure if that's a good idea.

Question: does the forward declaration of OriginBoundCertService
prevent us from defining this method inline in the header?

[rkn, 2011/08/09 20:32:05]

Yes, the forward declaration seems to prevent us from inlining the method.

+      net::OriginBoundCertService* origin_bound_cert_service) const;
+
   net::NetworkDelegate* network_delegate() const {
     return network_delegate_.get();
   }
 
   net::DnsCertProvenanceChecker* dns_cert_checker() const {
     return dns_cert_checker_.get();
   }
 
   net::ProxyService* proxy_service() const {
     return proxy_service_.get();
@@ skipping 53 matching lines @@
   mutable scoped_ptr<ProfileParams> profile_params_;
 
   // Member variables which are pointed to by the various context objects.
   mutable BooleanPrefMember enable_referrers_;
   mutable BooleanPrefMember clear_local_state_on_exit_;
   mutable BooleanPrefMember safe_browsing_enabled_;
 
   // Pointed to by URLRequestContext.
   mutable scoped_ptr<ChromeURLDataManagerBackend>
       chrome_url_data_manager_backend_;
+  mutable scoped_ptr<net::OriginBoundCertService> origin_bound_cert_service_;
   mutable scoped_ptr<net::NetworkDelegate> network_delegate_;
   mutable scoped_ptr<net::DnsCertProvenanceChecker> dns_cert_checker_;
   mutable scoped_ptr<net::ProxyService> proxy_service_;
   mutable scoped_ptr<net::URLRequestJobFactory> job_factory_;
 
   // Pointed to by ResourceContext.
   mutable scoped_refptr<webkit_database::DatabaseTracker> database_tracker_;
   mutable scoped_refptr<ChromeAppCacheService> appcache_service_;
   mutable scoped_refptr<ChromeBlobStorageContext> blob_storage_context_;
   mutable scoped_refptr<fileapi::FileSystemContext> file_system_context_;
@@ skipping 19 matching lines @@
   // Weak pointers to the request contexts. Only valid after LazyInitialize.
   // These are weak so that they don't hold a reference to the RequestContext,
   // because that holds a reference back to ProfileIOData.
   mutable base::WeakPtr<ChromeURLRequestContext>
       weak_extensions_request_context_;
 
   DISALLOW_COPY_AND_ASSIGN(ProfileIOData);
 };
 
 #endif  // CHROME_BROWSER_PROFILES_PROFILE_IO_DATA_H_