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

Side by Side Diff: components/safe_browsing/base_resource_throttle.h

Issue 2692273006: Refactor safebrowsing StartDisplayingBlockingPage (Closed)
Patch Set: Fix method description Created 3 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
1 // Copyright (c) 2017 The Chromium Authors. All rights reserved. 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 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 #ifndef COMPONENTS_SAFE_BROWSING_BASE_RESOURCE_THROTTLE_H_ 5 #ifndef COMPONENTS_SAFE_BROWSING_BASE_RESOURCE_THROTTLE_H_
6 #define COMPONENTS_SAFE_BROWSING_BASE_RESOURCE_THROTTLE_H_ 6 #define COMPONENTS_SAFE_BROWSING_BASE_RESOURCE_THROTTLE_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 protected: 79 protected:
80 BaseResourceThrottle( 80 BaseResourceThrottle(
81 const net::URLRequest* request, 81 const net::URLRequest* request,
82 content::ResourceType resource_type, 82 content::ResourceType resource_type,
83 scoped_refptr<SafeBrowsingDatabaseManager> 83 scoped_refptr<SafeBrowsingDatabaseManager>
84 database_manager, 84 database_manager,
85 scoped_refptr<BaseUIManager> ui_manager); 85 scoped_refptr<BaseUIManager> ui_manager);
86 86
87 ~BaseResourceThrottle() override; 87 ~BaseResourceThrottle() override;
88 88
89 // If our blocked resource is the main frame, this calls
90 // ContentSubresourceFilterDriverFactory's
91 // OnMainResourceMatchedSafeBrowsingBlacklist method.
92 static void NotifySubresourceFilterOfBlockedResource(
93 const security_interstitials::UnsafeResource& resource);
94
89 // Does nothing in the base class. Override this to destroy prerender contents 95 // Does nothing in the base class. Override this to destroy prerender contents
90 // in chrome. 96 // in chrome.
91 virtual void MaybeDestroyPrerenderContents( 97 virtual void MaybeDestroyPrerenderContents(
92 const content::ResourceRequestInfo* info); 98 const content::ResourceRequestInfo* info);
93 99
94 // Posts a task for StartDisplayingBlockingPage 100 // Posts a task for StartDisplayingBlockingPage
95 virtual void StartDisplayingBlockingPageHelper( 101 virtual void StartDisplayingBlockingPageHelper(
96 security_interstitials::UnsafeResource resource); 102 security_interstitials::UnsafeResource resource);
97 103
98 // Called by OnBlockingPageComplete when proceed == false. This removes the 104 // Called by OnBlockingPageComplete when proceed == false. This removes the
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
186 // http://crbug.com/660293 192 // http://crbug.com/660293
187 // URLs that timed out waiting for a SafeBrowsing reputation check. 193 // URLs that timed out waiting for a SafeBrowsing reputation check.
188 std::set<GURL> timed_out_urls_; 194 std::set<GURL> timed_out_urls_;
189 195
190 DISALLOW_COPY_AND_ASSIGN(BaseResourceThrottle); 196 DISALLOW_COPY_AND_ASSIGN(BaseResourceThrottle);
191 }; 197 };
192 198
193 } // namespace safe_browsing 199 } // namespace safe_browsing
194 200
195 #endif // COMPONENTS_SAFE_BROWSING_BASE_RESOURCE_THROTTLE_H_ 201 #endif // COMPONENTS_SAFE_BROWSING_BASE_RESOURCE_THROTTLE_H_
OLDNEW
« no previous file with comments | « chrome/browser/loader/safe_browsing_resource_throttle.cc ('k') | components/safe_browsing/base_resource_throttle.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698