Remove FileSystem runtime flag for chromium

Remove FileSystem runtime flag for chromium

This feature is in stable status since 2013. It should be fine to remove
this runtime flag.

The blink side patch is at: https://codereview.chromium.org/1117063005/

BUG=402536

[qiankun, 2015-05-04 06:31:32]

qiankun.miao@intel.com changed reviewers:
+ avi@chromium.org, mkwst@chromium.org

[qiankun, 2015-05-04 06:31:32]

PTAL.

[Mike West, 2015-05-04 14:20:14]

On 2015/05/04 at 06:31:32, qiankun.miao wrote:
> PTAL.

Please remove the usage in android_webview/lib/main/aw_main_delegate.cc as well.
Otherwise LGTM.

You'll need a //content OWNER to stamp it, however.

[Avi (use Gerrit), 2015-05-04 14:29:56]

lgtm

stampity stamp

[qiankun, 2015-05-04 15:19:04]

On 2015/05/04 14:20:14, Mike West (traveling. slow.) wrote:
> On 2015/05/04 at 06:31:32, qiankun.miao wrote:
> > PTAL.
> 
> Please remove the usage in android_webview/lib/main/aw_main_delegate.cc as
well.
> Otherwise LGTM.
> 
> You'll need a //content OWNER to stamp it, however.

Done.

[qiankun, 2015-05-04 15:38:16]

The CQ bit was checked by qiankun.miao@intel.com

[qiankun, 2015-05-04 15:38:16]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org,
mkwst@chromium.org
Link to the patchset: https://codereview.chromium.org/1117953005/#ps20001
(title: "remove all kDisableFileSystem")

[commit-bot: I haz the power, 2015-05-04 15:38:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1117953005/20001

[commit-bot: I haz the power, 2015-05-04 15:44:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-05-04 15:44:46]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[qiankun, 2015-05-04 17:19:24]

qiankun.miao@intel.com changed reviewers:
+ torne@chromium.org

[qiankun, 2015-05-04 17:19:24]

Torne@, could you do owner review for src/android_webview/?

[Torne, 2015-05-05 08:48:11]

not LGTM. The flag is used by WebView; please don't just remove where we set it.
The comment with the flag mentions possibly needing new APIs to allow the
embedder to control the filesystem API and I think we still need to consider
this. Also, have you tested that the filesystem API actually even works in
webview?

[qiankun, 2015-05-05 09:22:32]

On 2015/05/05 08:48:11, Torne wrote:
> not LGTM. The flag is used by WebView; please don't just remove where we set
it.
> The comment with the flag mentions possibly needing new APIs to allow the
> embedder to control the filesystem API and I think we still need to consider
> this. Also, have you tested that the filesystem API actually even works in
> webview?

Thanks for review. Yes, WebView is still using this flag to disable FileSystem.
Sorry, my mistake for omitting the check.