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

Unified Diff: chrome/browser/android/offline_pages/offline_page_request_job.cc

Issue 2786583002: chromeos: Check both original and absolute paths for file: scheme (Closed)
Patch Set: remove dcheck Created 3 years, 7 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/android/offline_pages/offline_page_request_job.cc
diff --git a/chrome/browser/android/offline_pages/offline_page_request_job.cc b/chrome/browser/android/offline_pages/offline_page_request_job.cc
index 32a66852d9b65813ba443787824be7aa475c5253..3401b11eb16b2f520a083a4c58514133afbeda8e 100644
--- a/chrome/browser/android/offline_pages/offline_page_request_job.cc
+++ b/chrome/browser/android/offline_pages/offline_page_request_job.cc
@@ -707,6 +707,13 @@ void OfflinePageRequestJob::OnOfflineRedirectAvailabe(
URLRequestJob::NotifyHeadersComplete();
}
+// Returns true to disable the file path checking for file: scheme in
+// URLRequestFileJob, that's not relevant for this class.
+bool OfflinePageRequestJob::CanAccessFile(const base::FilePath& original_path,
+ const base::FilePath& absolute_path) {
+ return true;
+}
+
void OfflinePageRequestJob::SetDelegateForTesting(
std::unique_ptr<Delegate> delegate) {
delegate_ = std::move(delegate);
« no previous file with comments | « chrome/browser/android/offline_pages/offline_page_request_job.h ('k') | chrome/browser/net/chrome_network_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698