REGRESSION(r119944): test http/tests/security/mixedContent/filesystem-url-in-iframe.html fails

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Szilard Ledan]

This test: http/tests/security/mixedContent/filesystem-url-in-iframe.html fails after r119944 on qt.

--- /ramdisk/qt-linux-64-release/build/layout-test-results/http/tests/security/mixedContent/filesystem-url-in-iframe-expected.txt 
+++ /ramdisk/qt-linux-64-release/build/layout-test-results/http/tests/security/mixedContent/filesystem-url-in-iframe-actual.txt 
@@ -1,8 +1,10 @@
-ALERT: PASS (1/2): File written
+CONSOLE MESSAGE: line 28: ReferenceError: Can't find variable: TEMPORARY
+CONSOLE MESSAGE: line 28: ReferenceError: Can't find variable: TEMPORARY
+FAIL: Timed out waiting for notifyDone to be called
 
 This tests that filesystem URLs created in a secure context are treated as secure origins. This test passes if the iframe renders PASS correctly, and no console warning appears.
 
 --------
 Frame: '<!--framePath //<!--frame0-->-->'
 --------
-PASS (2/2): File displayed
+

[Mike West]

I'll take a look.

[Csaba Osztrogonác]

Skipped on Qt to paint the bot green by http://trac.webkit.org/changeset/119964/trunk/LayoutTests/platform/qt/Skipped . Please unskip it with the proper fix.

[Mike West]

Actually, it looks like ENABLE(FILE_SYSTEM) is disabled on QT. I'll move the skip to the proper section of the Skipped file so that it can be reenabled if and when the port enables the feature.

[Mike West]

Created attachment 146813 [details]
Patch

[Mike West]

Comment on attachment 146813 [details]
Patch

Would you mind taking a look, and throwing it into the queue if you're happy?

[Csaba Osztrogonác]

Comment on attachment 146813 [details]
Patch

r=me, thanks for looking into the root of the problem.

[WebKit Review Bot]

Comment on attachment 146813 [details]
Patch

Clearing flags on attachment: 146813

Committed r119971: <http://trac.webkit.org/changeset/119971>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.