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

Unified Diff: chrome/browser/search/iframe_source.cc

Issue 2856093004: Generalize content::URLDataSource so that it can be used by the network service. (Closed)
Patch Set: review comments 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/search/iframe_source.cc
diff --git a/chrome/browser/search/iframe_source.cc b/chrome/browser/search/iframe_source.cc
index cae15fe43a0e36a3da2b8ee0bfb305a18d7a711a..49f0aa551f6215c77eb957faab356cfe7ba1934c 100644
--- a/chrome/browser/search/iframe_source.cc
+++ b/chrome/browser/search/iframe_source.cc
@@ -41,11 +41,14 @@ bool IframeSource::AllowCaching() const {
}
bool IframeSource::ShouldServiceRequest(
- const net::URLRequest* request) const {
- const std::string& path = request->url().path();
- return InstantIOContext::ShouldServiceRequest(request) &&
- request->url().SchemeIs(chrome::kChromeSearchScheme) &&
- request->url().host_piece() == GetSource() && ServesPath(path);
+ const GURL& url,
+ content::ResourceContext* resource_context,
+ int render_process_id) const {
+ const std::string& path = url.path();
+ return InstantIOContext::ShouldServiceRequest(url, resource_context,
+ render_process_id) &&
+ url.SchemeIs(chrome::kChromeSearchScheme) &&
+ url.host_piece() == GetSource() && ServesPath(path);
}
bool IframeSource::ShouldDenyXFrameOptions() const {

Powered by Google App Engine
This is Rietveld 408576698