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

Side by Side Diff: components/subresource_filter/content/browser/subresource_filter_safe_browsing_activation_throttle.cc

Issue 2645283007: Add the client for accessing Subresource Filter only list. (Closed)
Patch Set: feature Created 3 years, 8 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 (c) 2017 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 "components/subresource_filter/content/browser/subresource_filter_safe_ browsing_activation_throttle.h"
6
7 #include <vector>
8
9 #include "base/timer/timer.h"
10 #include "components/safe_browsing_db/v4_local_database_manager.h"
11 #include "components/subresource_filter/content/browser/content_subresource_filt er_driver_factory.h"
12 #include "content/public/browser/browser_thread.h"
13 #include "content/public/browser/navigation_handle.h"
14 #include "content/public/browser/web_contents.h"
15
16 namespace {
17
18 // Maximum time in milliseconds to wait for the Safe Browsing service to
19 // verify a URL. After this amount of time the outstanding check will be
20 // aborted, and the URL will be treated as if it didn't belong to the
21 // Subresource Filter only list.
22 constexpr base::TimeDelta kCheckURLTimeout = base::TimeDelta::FromSeconds(5);
23
24 } // namespace
25
26 namespace subresource_filter {
27
28 class SubresourceFilterSafeBrowsingActivationThrottle::SBDatabaseClient
29 : public safe_browsing::SafeBrowsingDatabaseManager::Client {
30 public:
31 SBDatabaseClient(
32 scoped_refptr<safe_browsing::SafeBrowsingDatabaseManager>
33 database_manager,
34 base::WeakPtr<SubresourceFilterSafeBrowsingActivationThrottle> throttle,
35 scoped_refptr<base::SingleThreadTaskRunner> io_task_runner)
36 : database_manager_(std::move(database_manager)),
37 throttle_(throttle),
38 io_task_runner_(io_task_runner) {}
39
40 ~SBDatabaseClient() override {
41 DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
42 database_manager_->CancelCheck(this);
43 }
44
45 void CheckUrlOnIO(const GURL& url) {
46 DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
47 DCHECK(!url.is_empty());
48 url_being_checked_ = url;
49 if (database_manager_->CheckUrlForSubresourceFilter(url, this)) {
50 OnCheckBrowseUrlResult(url, safe_browsing::SB_THREAT_TYPE_SAFE,
51 safe_browsing::ThreatMetadata());
52 return;
53 }
54 timer_.Start(FROM_HERE, kCheckURLTimeout, this,
55 &SubresourceFilterSafeBrowsingActivationThrottle::
56 SBDatabaseClient::OnCheckUrlTimeout);
57 }
58
59 void OnCheckBrowseUrlResult(
60 const GURL& url,
61 safe_browsing::SBThreatType threat_type,
62 const safe_browsing::ThreatMetadata& metadata) override {
63 DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
64 DCHECK_EQ(url_being_checked_, url);
65 timer_.Stop(); // Cancel the timeout timer.
66 io_task_runner_->PostTask(
67 FROM_HERE,
68 base::Bind(&SubresourceFilterSafeBrowsingActivationThrottle::
69 OnCheckUrlResultOnUI,
70 throttle_, url, threat_type, metadata.threat_pattern_type));
71 }
72
73 // Callback for when the safe browsing check has taken longer than
74 // kCheckURLTimeout.
75 void OnCheckUrlTimeout() {
76 DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
77 database_manager_->CancelCheck(this);
78
79 OnCheckBrowseUrlResult(url_being_checked_,
80 safe_browsing::SB_THREAT_TYPE_SAFE,
81 safe_browsing::ThreatMetadata());
82 }
83
84 private:
85 scoped_refptr<safe_browsing::SafeBrowsingDatabaseManager> database_manager_;
86
87 // Timer to abort the safe browsing check if it takes too long.
88 base::OneShotTimer timer_;
89 GURL url_being_checked_;
90
91 base::WeakPtr<SubresourceFilterSafeBrowsingActivationThrottle> throttle_;
92 scoped_refptr<base::SingleThreadTaskRunner> io_task_runner_;
93
94 DISALLOW_COPY_AND_ASSIGN(SBDatabaseClient);
95 };
96
97 SubresourceFilterSafeBrowsingActivationThrottle::
98 SubresourceFilterSafeBrowsingActivationThrottle(
99 content::NavigationHandle* handle,
100 scoped_refptr<safe_browsing::SafeBrowsingDatabaseManager>
101 database_manager)
102 : NavigationThrottle(handle),
103 io_task_runner_(content::BrowserThread::GetTaskRunnerForThread(
104 content::BrowserThread::IO)),
105 database_client_(
106 new SubresourceFilterSafeBrowsingActivationThrottle::SBDatabaseClient(
107 std::move(database_manager),
108 AsWeakPtr(),
109 base::ThreadTaskRunnerHandle::Get()),
110 base::OnTaskRunnerDeleter(io_task_runner_)) {}
111
112 SubresourceFilterSafeBrowsingActivationThrottle::
113 ~SubresourceFilterSafeBrowsingActivationThrottle() {}
114
115 content::NavigationThrottle::ThrottleCheckResult
116 SubresourceFilterSafeBrowsingActivationThrottle::WillProcessResponse() {
117 io_task_runner_->PostTask(
118 FROM_HERE, base::Bind(&SubresourceFilterSafeBrowsingActivationThrottle::
119 SBDatabaseClient::CheckUrlOnIO,
120 base::Unretained(database_client_.get()),
121 navigation_handle()->GetURL()));
122 return content::NavigationThrottle::ThrottleCheckResult::DEFER;
123 }
124
125 void SubresourceFilterSafeBrowsingActivationThrottle::OnCheckUrlResultOnUI(
126 const GURL& url,
127 safe_browsing::SBThreatType threat_type,
128 safe_browsing::ThreatPatternType pattern_type) {
129 content::WebContents* web_contents = navigation_handle()->GetWebContents();
130 if (web_contents) {
131 using subresource_filter::ContentSubresourceFilterDriverFactory;
132 ContentSubresourceFilterDriverFactory* driver_factory =
133 ContentSubresourceFilterDriverFactory::FromWebContents(web_contents);
134 DCHECK(driver_factory);
135
136 driver_factory->OnMainResourceMatchedSafeBrowsingBlacklist(
137 url, std::vector<GURL>(), threat_type, pattern_type);
138 }
139 // TODO(https://crbug.com/704508): We should measure the delay introduces by
140 // this check. Similarly, as it's done the Safe Browsing Resource throttle.
141 navigation_handle()->Resume();
142 }
143
144 } // namespace subresource_filter
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698