Bug 146697 - [Windows] LayoutTests/http/tests/security/contentSecurityPolicy/userAgentShadowDOM/allow-audio.html fails
Summary: [Windows] LayoutTests/http/tests/security/contentSecurityPolicy/userAgentShad...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows 8
: P2 Normal
Assignee: Nobody
URL:
Keywords: LayoutTestFailure, PlatformOnly
Depends on: 144830
Blocks:
  Show dependency tree / graph
 
Reported: 2015-07-07 15:49 PDT by Daniel Bates
Modified: 2015-07-08 11:48 PDT (History)
2 users (show)

See Also:

Attachments
[Patch] Speculative patch (23.35 KB, patch)
2015-07-07 15:51 PDT, Daniel Bates 		bfulgham: review-
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2015-07-07 15:49:59 PDT 
The test LayoutTests/http/tests/security/contentSecurityPolicy/userAgentShadowDOM/allow-audio.html fails on Windows with diff:
[[
--- /home/buildbot/slave/win-release-tests/build/layout-test-results/http/tests/security/contentSecurityPolicy/userAgentShadowDOM/allow-audio-expected.txt
+++ /home/buildbot/slave/win-release-tests/build/layout-test-results/http/tests/security/contentSecurityPolicy/userAgentShadowDOM/allow-audio-actual.txt
@@ -1,4 +1,4 @@
 Tests that a HTML audio element, in a user agent shadow tree, is allowed to load when the page has CSP policy: media-src 'none'.
 
-PASS did load audio metadata.
+FAIL did not load audio metadata.
]]
<https://build.webkit.org/results/Apple%20Win%207%20Release%20(Tests)/r186414%20(52833)/http/tests/security/contentSecurityPolicy/userAgentShadowDOM/allow-audio-diff.txt>

See <https://build.webkit.org/results/Apple%20Win%207%20Release%20(Tests)/r186414%20(52833)/results.html>
Comment 1 Daniel Bates 2015-07-07 15:50:31 PDT 
The test was skipped in <http://trac.webkit.org/changeset/186447>.
Comment 2 Daniel Bates 2015-07-07 15:51:19 PDT 
Created attachment 256326 [details]
[Patch] Speculative patch
Comment 3 Brent Fulgham 2015-07-07 16:43:08 PDT 
Comment on attachment 256326 [details]
[Patch] Speculative patch

View in context: https://bugs.webkit.org/attachment.cgi?id=256326&action=review

> LayoutTests/http/tests/security/contentSecurityPolicy/userAgentShadowDOM/allow-audio.html:39
> +    audio.src = "resources/crow.wav";

Amazingly, WAV files won't work on Windows (at least, not Apple Windows) because we don't support WAV file decoding. Can you use an MP3 or something similar?
Comment 4 Brent Fulgham 2015-07-07 17:05:42 PDT 
The actual problem here is that "internals.ensureUserAgentShadowRoot" is not defined on Windows, at least for the element being queried. I'm debugging this now.
Comment 5 Brent Fulgham 2015-07-08 11:48:28 PDT 
The original AIF-based test runs fine locally on Release and Debug builds. So I think the test failure is purely a test machine configuration issue, and this patch is not necessary.