Fix the issue that Quick View doesn't load images.

Fix the issue that Quick View doesn't load images.

The files_safe_media_webview_content.js, which is loaded inside webview,
should be whitelisted in accessible_resources.

Quick view had happened to be working before 2768863003 was checked in.
After the bug fix on webview is submitted, this proper fix is needed for
Quick view to work.

BUG=706537
TEST=manually tested Quick View opens an image file.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2797893003
Cr-Commit-Position: refs/heads/master@{#461935}
Committed: https://chromium.googlesource.com/chromium/src/+/c433f3648b6a43e43318b0ba72717fd42304b76e

[oka, 2017-04-05 01:24:37]

Description was changed from

==========
Fix the issue that Quick View doens't load images.

The files_safe_media_webview_content.js, which is loaded inside webview,
should be whitelisted in accessible_resources.

Quick view had happened to be working before 2768863003 was checked in.
After the bug fix on webview is submitted, this proper fix is needed for
Quick view to work.

BUG=706537
==========

to

==========
Fix the issue that Quick View doens't load images.

The files_safe_media_webview_content.js, which is loaded inside webview,
should be whitelisted in accessible_resources.

Quick view had happened to be working before 2768863003 was checked in.
After the bug fix on webview is submitted, this proper fix is needed for
Quick view to work.

BUG=706537
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[oka, 2017-04-05 01:25:44]

Description was changed from

==========
Fix the issue that Quick View doens't load images.

The files_safe_media_webview_content.js, which is loaded inside webview,
should be whitelisted in accessible_resources.

Quick view had happened to be working before 2768863003 was checked in.
After the bug fix on webview is submitted, this proper fix is needed for
Quick view to work.

BUG=706537
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Fix the issue that Quick View doens't load images.

The files_safe_media_webview_content.js, which is loaded inside webview,
should be whitelisted in accessible_resources.

Quick view had happened to be working before 2768863003 was checked in.
After the bug fix on webview is submitted, this proper fix is needed for
Quick view to work.

BUG=706537
TEST=manually tested Quick View opens an image file.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[oka, 2017-04-05 01:25:45]

oka@chromium.org changed reviewers:
+ fukino@chromium.org

[oka, 2017-04-05 01:25:52]

PTAL.

[oka, 2017-04-05 01:26:06]

Description was changed from

==========
Fix the issue that Quick View doens't load images.

The files_safe_media_webview_content.js, which is loaded inside webview,
should be whitelisted in accessible_resources.

Quick view had happened to be working before 2768863003 was checked in.
After the bug fix on webview is submitted, this proper fix is needed for
Quick view to work.

BUG=706537
TEST=manually tested Quick View opens an image file.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Fix the issue that Quick View doesn't load images.

The files_safe_media_webview_content.js, which is loaded inside webview,
should be whitelisted in accessible_resources.

Quick view had happened to be working before 2768863003 was checked in.
After the bug fix on webview is submitted, this proper fix is needed for
Quick view to work.

BUG=706537
TEST=manually tested Quick View opens an image file.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[fukino, 2017-04-05 01:28:15]

lgtm with a nit.

https://codereview.chromium.org/2797893003/diff/1/ui/file_manager/file_manage...
File ui/file_manager/file_manager/manifest.json (right):

https://codereview.chromium.org/2797893003/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/manifest.json:58:
"foreground/elements/files_safe_media_webview_content.js"
nit: Sort alphabetically.

[oka, 2017-04-05 01:29:15]

On 2017/04/05 01:28:15, fukino wrote:
> lgtm with a nit.
> 
>
https://codereview.chromium.org/2797893003/diff/1/ui/file_manager/file_manage...
> File ui/file_manager/file_manager/manifest.json (right):
> 
>
https://codereview.chromium.org/2797893003/diff/1/ui/file_manager/file_manage...
> ui/file_manager/file_manager/manifest.json:58:
> "foreground/elements/files_safe_media_webview_content.js"
> nit: Sort alphabetically.

Done.

[oka, 2017-04-05 01:29:21]

.

[oka, 2017-04-05 01:29:39]

The CQ bit was checked by oka@chromium.org

[oka, 2017-04-05 01:29:40]

The patchset sent to the CQ was uploaded after l-g-t-m from fukino@chromium.org
Link to the patchset: https://codereview.chromium.org/2797893003/#ps20001
(title: ".")

[commit-bot: I haz the power, 2017-04-05 01:30:18]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-05 01:56:56]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491355779817610,
"parent_rev": "43fdb1771d2b601fa5bb604c2decf6eccf98f2c2", "commit_rev":
"c433f3648b6a43e43318b0ba72717fd42304b76e"}

[commit-bot: I haz the power, 2017-04-05 01:57:47]

Description was changed from

==========
Fix the issue that Quick View doesn't load images.

The files_safe_media_webview_content.js, which is loaded inside webview,
should be whitelisted in accessible_resources.

Quick view had happened to be working before 2768863003 was checked in.
After the bug fix on webview is submitted, this proper fix is needed for
Quick view to work.

BUG=706537
TEST=manually tested Quick View opens an image file.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Fix the issue that Quick View doesn't load images.

The files_safe_media_webview_content.js, which is loaded inside webview,
should be whitelisted in accessible_resources.

Quick view had happened to be working before 2768863003 was checked in.
After the bug fix on webview is submitted, this proper fix is needed for
Quick view to work.

BUG=706537
TEST=manually tested Quick View opens an image file.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2797893003
Cr-Commit-Position: refs/heads/master@{#461935}
Committed:
https://chromium.googlesource.com/chromium/src/+/c433f3648b6a43e43318b0ba7271...
==========

[commit-bot: I haz the power, 2017-04-05 01:57:50]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/c433f3648b6a43e43318b0ba7271...