Bug 153697 - [GTK] Some new CSP tests are failing with one-space difference in output
Summary: [GTK] Some new CSP tests are failing with one-space difference in output
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-29 21:00 PST by Michael Catanzaro
Modified: 2016-02-01 15:55 PST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2016-01-29 21:00:05 PST
The following layout tests have been failing for GTK since they were added in r195367:

http/tests/security/contentSecurityPolicy/object-src-applet-archive-codebase.html [ Failure ]
http/tests/security/contentSecurityPolicy/object-src-applet-archive.html [ Failure ]
http/tests/security/contentSecurityPolicy/object-src-applet-code-codebase.html [ Failure ]
http/tests/security/contentSecurityPolicy/object-src-applet-code.html [ Failure ]

The only difference from the expected output is that there is one extra space character that doesn't exist on the other ports:

-This test passes if there is a console message saying the plugin was blocked. The test must be run manually; ContentShell doesn't support applet in DumpRenderTree mode. :(
+This test passes if there is a console message saying the plugin was blocked. The test must be run manually; ContentShell doesn't support applet in DumpRenderTree mode. :( 

(Select those lines to note the whitespace difference at the end of the lines.)

I'm tempted to just add expected results with the space, but I have no clue where the space is coming from; it doesn't seem right. So I will instead mark these as failing for now.

Also, I'm not pleased that (a) this is a manual test outside of the ManualTests directory, and (b) there is no console message saying that the plugin was blocked even when running the test manually (at least in 2.10.4; maybe that was one of the recent fixes?).
Comment 1 Daniel Bates 2016-01-31 18:56:18 PST
We should just remove the tests for now. I inadvertently imported these manual tests from Blink. We need to convert this tests to DumpRenderTree/WebKitTestRunner tests (if possible) or move them to the top-level ManualTests directory.
Comment 2 Daniel Bates 2016-02-01 15:55:29 PST
(In reply to comment #1)
> We should just remove the tests for now. I inadvertently imported these
> manual tests from Blink. We need to convert this tests to
> DumpRenderTree/WebKitTestRunner tests (if possible) or move them to the
> top-level ManualTests directory.

Filed bug #153776 to convert these tests to DumpRenderTree/WebKitTestRunner tests.