chromeos: Check both original and absolute paths for file: scheme

chrome/browser/android/offline_pages/offline_page_request_job.cc

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;

[jianli, 2017/05/26 18:52:52]

Though it is not relevant, will returning false be a bit safer?

[mmenke, 2017/05/26 19:00:04]

Wouldn't that break this class entirely?  This subclasses URLRequestFileJob, and
requires access to whatever file it's mapping a request to.  If it can't access
that file, it won't work.

+}
+
 void OfflinePageRequestJob::SetDelegateForTesting(
     std::unique_ptr<Delegate> delegate) {
   delegate_ = std::move(delegate);