File App tests include resources that violate CSP |
|||
Issue descriptionThere are a number of File Manager API and Video Player tests that are missing CSPs for a base test class they use.
,
Jul 10
,
Jul 11
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cd62293b3cca951905963157dacf832c4b2dcd55 commit cd62293b3cca951905963157dacf832c4b2dcd55 Author: David Bertoni <dbertoni@chromium.org> Date: Wed Jul 11 15:16:50 2018 [APPS] Add content security policy to the ChromeOS video player app. The Video Player app has a test, VideoPlayerBrowserTest.OpenSingleVideoOnDrive, that loads a script from ui/file_manager/integration_tests. This script would normally be blocked by the platform app's CSP. Update the app to use an explicit CSP allowing the extension's resource. Bug: 861815 Cq-Include-Trybots: luci.chromium.try:closure_compilation Change-Id: Idf10b0eea4e284ea4c3f0c624dc05c5a340fc4ed Reviewed-on: https://chromium-review.googlesource.com/1132314 Commit-Queue: David Bertoni <dbertoni@chromium.org> Reviewed-by: Devlin <rdevlin.cronin@chromium.org> Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org> Cr-Commit-Position: refs/heads/master@{#574167} [modify] https://crrev.com/cd62293b3cca951905963157dacf832c4b2dcd55/ui/file_manager/video_player/manifest.json
,
Jul 11
|
|||
►
Sign in to add a comment |
|||
Comment 1 by bugdroid1@chromium.org
, Jul 9