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

net/base/network_delegate_impl.h

 // 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.
 
 #ifndef NET_BASE_NETWORK_DELEGATE_IMPL_H_
 #define NET_BASE_NETWORK_DELEGATE_IMPL_H_
 
 #include <stdint.h>
 
 #include "base/strings/string16.h"
@@ skipping 164 matching lines @@
                        const CookieList& cookie_list) override;
 
   // Called when a cookie is set to allow the network delegate to block access
   // to the cookie. This method will never be invoked when
   // LOAD_DO_NOT_SAVE_COOKIES is specified.
   bool OnCanSetCookie(const URLRequest& request,
                       const std::string& cookie_line,
                       CookieOptions* options) override;
 
   // Called when a file access is attempted to allow the network delegate to
-  // allow or block access to the given file path.  Returns true if access is
-  // allowed.
+  // allow or block access to the given file path, provided in the original
+  // and absolute forms (i.e. symbolic link is resolved). It's up to the sub
+  // class to decide which path to use for checking. Returns true if access
+  // is allowed.

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

same comment as in other file...  Or better, just remove all comments from this
file, as they're redundant with the other one.  If you prefer not to do that in
this CL, that's fine.

[satorux1, 2017/04/21 08:05:37]

Let me do it in a separate patch.

   bool OnCanAccessFile(const URLRequest& request,
-                       const base::FilePath& path) const override;
+                       const base::FilePath& original_path,
+                       const base::FilePath& absolute_path) const override;
 
   // Returns true if the given |url| has to be requested over connection that
   // is not tracked by the server. Usually is false, unless user privacy
   // settings block cookies from being get or set.
   bool OnCanEnablePrivacyMode(
       const GURL& url,
       const GURL& first_party_for_cookies) const override;
 
   // Returns true if the embedder has enabled experimental cookie features.
   bool OnAreExperimentalCookieFeaturesEnabled() const override;
 
   // Called when the |referrer_url| for requesting |target_url| during handling
   // of the |request| is does not comply with the referrer policy (e.g. a
   // secure referrer for an insecure initial target).
   // Returns true if the request should be cancelled. Otherwise, the referrer
   // header is stripped from the request.
   bool OnCancelURLRequestWithPolicyViolatingReferrerHeader(
       const URLRequest& request,
       const GURL& target_url,
       const GURL& referrer_url) const override;
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_NETWORK_DELEGATE_IMPL_H_