Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(38)

Side by Side Diff: chrome/browser/safe_browsing/certificate_reporting_service_factory.cc

Issue 2543523002: Implement main CertificateReportingService code and add unit tests. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/safe_browsing/certificate_reporting_service.h" 5 #include "chrome/browser/safe_browsing/certificate_reporting_service.h"
6 #include "chrome/browser/safe_browsing/certificate_reporting_service_factory.h" 6 #include "chrome/browser/safe_browsing/certificate_reporting_service_factory.h"
7 #include "components/keyed_service/content/browser_context_dependency_manager.h" 7 #include "components/keyed_service/content/browser_context_dependency_manager.h"
8 8
9 // static 9 // static
10 CertificateReportingServiceFactory* 10 CertificateReportingServiceFactory*
(...skipping 11 matching lines...) Expand all
22 22
23 CertificateReportingServiceFactory::CertificateReportingServiceFactory() 23 CertificateReportingServiceFactory::CertificateReportingServiceFactory()
24 : BrowserContextKeyedServiceFactory( 24 : BrowserContextKeyedServiceFactory(
25 "cert_reporting::Factory", 25 "cert_reporting::Factory",
26 BrowserContextDependencyManager::GetInstance()) {} 26 BrowserContextDependencyManager::GetInstance()) {}
27 27
28 CertificateReportingServiceFactory::~CertificateReportingServiceFactory() {} 28 CertificateReportingServiceFactory::~CertificateReportingServiceFactory() {}
29 29
30 KeyedService* CertificateReportingServiceFactory::BuildServiceInstanceFor( 30 KeyedService* CertificateReportingServiceFactory::BuildServiceInstanceFor(
31 content::BrowserContext* profile) const { 31 content::BrowserContext* profile) const {
32 return new CertificateReportingService(); 32 return nullptr;
Jialiu Lin 2016/11/30 22:18:06 may be add a TODO?
meacer 2016/11/30 23:39:40 Done.
33 } 33 }
34 34
35 content::BrowserContext* 35 content::BrowserContext*
36 CertificateReportingServiceFactory::GetBrowserContextToUse( 36 CertificateReportingServiceFactory::GetBrowserContextToUse(
37 content::BrowserContext* context) const { 37 content::BrowserContext* context) const {
38 return context; 38 return context;
39 } 39 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698