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

extensions/shell/browser/shell_network_delegate.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "extensions/shell/browser/shell_network_delegate.h"
 
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/resource_request_info.h"
 #include "extensions/browser/api/web_request/web_request_api.h"
 #include "extensions/browser/extension_system.h"
@@ skipping 91 matching lines @@
     net::URLRequest* request) {
   ExtensionWebRequestEventRouter::GetInstance()->OnURLRequestDestroyed(
       browser_context_, request);
 }
 
 void ShellNetworkDelegate::OnPACScriptError(
     int line_number,
     const base::string16& error) {
 }
 
+bool ShellNetworkDelegate::OnCanAccessFile(
+    const net::URLRequest& request,
+    const base::FilePath& original_path,
+    const base::FilePath& absolute_path) const {
+  return true;

[mmenke, 2017/04/18 17:24:45]

Why change behavior here?

[satorux1, 2017/04/19 07:22:22]

Without this, extensions tests failed, because access to some test files got
rejected by the check in URLRequestFileJob. I need to double-check but it seemed
that the code here is user for testing purpose.

[satorux1, 2017/04/21 06:35:10]

this turned out to be no longer necessary, due to the change
inextension_protocols.cc 

+}
+
 net::NetworkDelegate::AuthRequiredResponse
 ShellNetworkDelegate::OnAuthRequired(
     net::URLRequest* request,
     const net::AuthChallengeInfo& auth_info,
     const AuthCallback& callback,
     net::AuthCredentials* credentials) {
   return ExtensionWebRequestEventRouter::GetInstance()->OnAuthRequired(
       browser_context_, extension_info_map_.get(), request, auth_info, callback,
       credentials);
 }
 
 }  // namespace extensions