Fix security errors when accessing files and volumes in Files.app V2 due to using unregistered hand…

Fix security errors when accessing files and volumes in Files.app V2 due to using unregistered handlers.

The security errors were caused by revoking permissions to a file, directory or a root directory of a volume caused by launching Files.app using a handler not declared in the manifest. This patch fixes this issue by declaring all used handlers in the manifests and disallowing of calling undeclared handlers.

TEST=Check these bugs' repro steps.
BUG=233641, 237134, 236743, 238239, 234056
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=198744

[mtomasz, 2013-05-07 08:07:24]

@hashimoto: PTAL. Thanks.

[hashimoto, 2013-05-07 09:17:19]

lgtm

https://codereview.chromium.org/14864004/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/manifest_new_ui.json (right):

https://codereview.chromium.org/14864004/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/manifest_new_ui.json:35: // Automatically
open a volume and later close Files.app when unmounted.
nit: s/open/opens/? the same goes for others and manifest.json.

[mtomasz, 2013-05-07 10:16:15]

https://codereview.chromium.org/14864004/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/manifest_new_ui.json (right):

https://codereview.chromium.org/14864004/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/manifest_new_ui.json:35: // Automatically
open a volume and later close Files.app when unmounted.
On 2013/05/07 09:17:20, hashimoto wrote:
> nit: s/open/opens/? the same goes for others and manifest.json.

Done.

[commit-bot: I haz the power, 2013-05-07 10:16:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/14864004/5001

[commit-bot: I haz the power, 2013-05-07 10:23:43]

Sorry for I got bad news for ya.
Compile failed with a clobber build on ios_dbg_simulator.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-05-07 10:25:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/14864004/5001

[commit-bot: I haz the power, 2013-05-07 10:28:06]

Sorry for I got bad news for ya.
Compile failed with a clobber build on ios_dbg_simulator.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-05-07 10:29:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/14864004/5001

[commit-bot: I haz the power, 2013-05-07 16:55:04]

Change committed as 198744