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

Side by Side Diff: chrome/browser/safe_browsing/incident_reporting/module_load_analyzer.cc

Issue 1643573002: Add a ModuleLoadAnalyzer which checks modules against a whitelist (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Only enable for extended reporting AND feature flag. Add unit test. Created 4 years, 10 months 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "chrome/browser/safe_browsing/incident_reporting/module_load_analyzer.h "
6
7 #include "build/build_config.h"
8 #include "chrome/browser/browser_process.h"
9 #include "chrome/browser/safe_browsing/incident_reporting/incident_receiver.h"
10 #include "chrome/browser/safe_browsing/safe_browsing_service.h"
11
12 namespace safe_browsing {
13
14 #if defined(OS_WIN)
robertshield 2016/02/05 21:20:32 would prefer to have this in the _win.cc file. Thi
proberge 2016/02/05 22:26:10 Done.
15 // Enables automatic tab discarding, when the system is in low memory state.
16 const base::Feature kIncidentReportingModuleLoadAnalysis{
17 "IncidentReportingModuleLoadAnalysis", base::FEATURE_DISABLED_BY_DEFAULT};
18 #endif // defined(OS_WIN)
19
20 ModuleLoadAnalyzer::ModuleLoadAnalyzer(
21 const scoped_refptr<SafeBrowsingDatabaseManager>& database_manager)
22 : incident_receiver_(nullptr),
23 database_manager_(database_manager),
24 abort_reporting_(false) {
25 DCHECK(database_manager);
26 #if defined(OS_WIN)
27 if (base::FeatureList::IsEnabled(kIncidentReportingModuleLoadAnalysis)) {
28 scoped_refptr<SafeBrowsingService> safe_browsing_service(
29 g_browser_process->safe_browsing_service());
30
31 if (safe_browsing_service) {
32 safe_browsing_service
33 ->RegisterExtendedReportingOnlyDelayedAnalysisCallback(
34 base::Bind(&ModuleLoadAnalyzer::VerifyModuleLoadState,
35 base::Unretained(this)));
36 }
37 }
38 #endif // defined(OS_WIN)
39 }
40
41 ModuleLoadAnalyzer::~ModuleLoadAnalyzer() {
42 abort_reporting_ = true;
43 }
44
45 #if !defined(OS_WIN)
46 void ModuleLoadAnalyzer::GetLoadedSuspiciousModulesOnIOThread() {}
47
48 void ModuleLoadAnalyzer::ReportIncidentsForSuspiciousModules(
49 scoped_ptr<std::set<base::string16>> module_names) {}
50
51 bool ModuleLoadAnalyzer::VerifyModuleLoadState(
52 scoped_ptr<IncidentReceiver> incident_receiver) {
53 return false;
54 }
55 #endif // !defined(OS_WIN)
56
57 } // namespace safe_browsing
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698