Bug 54119 - [GTK] The Acid2 test fails
Summary: [GTK] The Acid2 test fails
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P3 Normal
Assignee: Nobody
URL:
Keywords: Gtk
Depends on:
Blocks:
 
Reported: 2011-02-09 10:24 PST by Martin Robinson
Modified: 2012-12-13 01:18 PST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Robinson 2011-02-09 10:24:39 PST
This can be observed by looking at the pixel results of fast/css/acid2-pixel.html. I've attached the result to this bug. In the pixel output the eyes portion of the face is rendered as a long red rectangle.
Comment 1 Sergio Villar Senin 2011-04-07 12:32:29 PDT
I have just comitted 48acff9e9e764931d3ea4f1119f4522451de3706 in libsoup. This will fix the acid2 pixel test issue.

Basically the problem was that we were not decoding percent escape triplets in base64 data: URLs. That's why g_base64_decode() with such kind of data was always returning an error.

I guess we should generate new libsoup packages for bots and then close this bug.
Comment 2 Philippe Normand 2011-06-29 04:46:02 PDT
I skipped fast/css/acid2.html and fast/css/acid2-pixel.html
in r90009.

Any news about this bug? I locally updated libsoup in my 64-Bits debug build and could still see those 2 fail while the 64-Bits bots run them fine. I'm confused :(
Comment 3 Zan Dobersek 2012-05-05 09:23:12 PDT
The following tests were found to be passing consistently after moving from using the Skipped list to using test_expectations.txt:
	fast/css/acid2-pixel.html
	fast/css/acid2.html
	http/tests/misc/acid2-pixel.html

Their expectations were removed in r116122[1] (covered by bug #85591). Tests that are covered by this bug and are still failing are the following:
	http/tests/misc/acid2.html

1: http://trac.webkit.org/changeset/116122
Comment 4 Martin Robinson 2012-12-13 01:18:43 PST
I think the test is passing now.