Restrict file access on android.

Restrict file access on android.

Only files in external storage directory are allowed.

BUG=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=155121

[willchan no longer on Chromium, 2012-09-06 00:20:15]

http://codereview.chromium.org/10905114/diff/1/chrome/browser/net/chrome_netw...
File chrome/browser/net/chrome_network_delegate.cc (right):

http://codereview.chromium.org/10905114/diff/1/chrome/browser/net/chrome_netw...
chrome/browser/net/chrome_network_delegate.cc:362: #if defined(OS_CHROMEOS) ||
defined(OS_ANDROID)
How about doing:

#if !defined(OS_CHROME) && !defined(OS_ANDROID)
  return true;
#else

http://codereview.chromium.org/10905114/diff/1/chrome/browser/net/chrome_netw...
chrome/browser/net/chrome_network_delegate.cc:380: #endif
Combine the #endif and #if into a #elif?

[nilesh, 2012-09-06 00:26:14]

PTAL.

http://codereview.chromium.org/10905114/diff/1/chrome/browser/net/chrome_netw...
File chrome/browser/net/chrome_network_delegate.cc (right):

http://codereview.chromium.org/10905114/diff/1/chrome/browser/net/chrome_netw...
chrome/browser/net/chrome_network_delegate.cc:362: #if defined(OS_CHROMEOS) ||
defined(OS_ANDROID)
On 2012/09/06 00:20:15, willchan wrote:
> How about doing:
> 
> #if !defined(OS_CHROME) && !defined(OS_ANDROID)
>   return true;
> #else

Done.

http://codereview.chromium.org/10905114/diff/1/chrome/browser/net/chrome_netw...
chrome/browser/net/chrome_network_delegate.cc:380: #endif
On 2012/09/06 00:20:15, willchan wrote:
> Combine the #endif and #if into a #elif?

Done.

[willchan no longer on Chromium, 2012-09-06 00:33:16]

lgtm

[commit-bot: I haz the power, 2012-09-06 00:35:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nileshagrawal@chromium.org/10905114/2003

[commit-bot: I haz the power, 2012-09-06 04:28:06]

Change committed as 155121