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

Unified Diff: chromecast/browser/cast_network_delegate.cc

Issue 2786583002: chromeos: Check both original and absolute paths for file: scheme (Closed)
Patch Set: address comments 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: chromecast/browser/cast_network_delegate.cc
diff --git a/chromecast/browser/cast_network_delegate.cc b/chromecast/browser/cast_network_delegate.cc
index 3e004a0a53934403e3944e9070613a925c058a86..3be211f596f5b412af864f7c41dcbc7bf207ede9 100644
--- a/chromecast/browser/cast_network_delegate.cc
+++ b/chromecast/browser/cast_network_delegate.cc
@@ -18,14 +18,16 @@ CastNetworkDelegate::CastNetworkDelegate() {
CastNetworkDelegate::~CastNetworkDelegate() {
}
-bool CastNetworkDelegate::OnCanAccessFile(const net::URLRequest& request,
- const base::FilePath& path) const {
+bool CastNetworkDelegate::OnCanAccessFile(
+ const net::URLRequest& request,
+ const base::FilePath& original_path,
+ const base::FilePath& absolute_path) const {
if (base::CommandLine::ForCurrentProcess()->
HasSwitch(switches::kEnableLocalFileAccesses)) {
return true;
}
- LOG(WARNING) << "Could not access file " << path.value()
+ LOG(WARNING) << "Could not access file " << original_path.value()
<< ". All file accesses are forbidden.";
return false;
}

Powered by Google App Engine
This is Rietveld 408576698