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

Unified Diff: extensions/shell/browser/shell_network_delegate.cc

Issue 2786583002: chromeos: Check both original and absolute paths for file: scheme (Closed)
Patch Set: disable checks in sub classes 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: extensions/shell/browser/shell_network_delegate.cc
diff --git a/extensions/shell/browser/shell_network_delegate.cc b/extensions/shell/browser/shell_network_delegate.cc
index b1c17380c436613fce44b1dd5a57bfd6a63cc191..fd1ed156378d2c85052680eadb0190f6d0479ba9 100644
--- a/extensions/shell/browser/shell_network_delegate.cc
+++ b/extensions/shell/browser/shell_network_delegate.cc
@@ -109,6 +109,13 @@ void ShellNetworkDelegate::OnPACScriptError(
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 acc
satorux1 2017/04/21 06:35:10 this turned out to be no longer necessary, due to
+}
+
net::NetworkDelegate::AuthRequiredResponse
ShellNetworkDelegate::OnAuthRequired(
net::URLRequest* request,

Powered by Google App Engine
This is Rietveld 408576698