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"
 #include "base/callback.h"
 #include "base/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "base/synchronization/lock.h"
 #include "chrome/browser/net/chrome_url_request_context.h"
 #include "chrome/browser/prefs/pref_member.h"
 #include "content/browser/resource_context.h"
 #include "net/base/cookie_monster.h"
+#include "net/base/origin_bound_cert_service.h"
 
 class CommandLine;
 class ChromeAppCacheService;
 class ChromeBlobStorageContext;
 class DesktopNotificationService;
 class ExtensionInfoMap;
 class HostContentSettingsMap;
 class HostZoomMap;
 class IOThread;
 class Profile;
@@ skipping 169 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();
   }
 
+  net::OriginBoundCertService* origin_bound_cert_service() const {
+    return origin_bound_cert_service_.get();
+  }

[rkn, 2011/08/06 00:03:25]

This breaks consistency because none of the others has a setter.

[willchan no longer on Chromium, 2011/08/06 14:20:49]

This is fine, although I'm not sure you need the accessor. If the subclass is
constructing and setting it, then it already has the pointer. The accessors are
necessary so the subclasses can set the member into the contexts, since the
other members are constructed by the base class.

[rkn, 2011/08/07 22:51:15]

Done.

+  void set_origin_bound_cert_service(
+      net::OriginBoundCertService* origin_bound_cert_service) const {
+    origin_bound_cert_service_.reset(origin_bound_cert_service);
+  }
+
   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_