Bug 7276 - Most W3C padding related tests fail
Summary: Most W3C padding related tests fail
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL: http://www.w3.org/Style/CSS/Test/CSS2...
Keywords:
Depends on:
Blocks:
 
Reported: 2006-02-15 08:55 PST by Diego Miguel Virasoro
Modified: 2022-06-09 21:46 PDT (History)
8 users (show)

See Also:


Attachments
patch (29.73 KB, patch)
2008-05-08 07:44 PDT, Anatoli Papirovski
mitz: review-
Details | Formatted Diff | Diff
A test case without padding (992 bytes, text/html)
2008-05-08 08:33 PDT, mitz
no flags Details
test case for "spanned" string wrapping (207 bytes, text/html)
2008-05-08 23:04 PDT, Robert Blaut
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Miguel Virasoro 2006-02-15 08:55:40 PST
I have tested Safari using the latest webkit (15th Feb) and it fails the following tests regarding paddings:

PADDING TOP
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5506-ipadn-t-00-b-a.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5506-ipadn-t-01-b-a.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5506-ipadn-t-02-b-a.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5506-padn-t-00-b-a.htm

PADDING RIGHT
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5507-ipadn-r-00-b-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5507-ipadn-r-01-b-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5507-ipadn-r-02-b-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5507-ipadn-r-03-b-a.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5507-ipadn-r-04-b-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5507-padn-r-00-c-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5507-padn-r-01-c-a.htm

PADDING BOTTOM
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5508-ipadn-b-00-b-a.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5508-ipadn-b-01-f-a.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5508-ipadn-b-02-b-a.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5508-ipadn-b-03-b-a.htm

PADDING RIGHT
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5509-ipadn-l-00-b-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5509-ipadn-l-01-b-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5509-ipadn-l-02-b-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5509-ipadn-l-03-b-a.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5509-ipadn-l-04-f-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5509-padn-l-00-b-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5509-padn-l-01-b-a.htm

PADDING
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5510-ipadn-00-b-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5510-padn-00-b-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5510-padn-01-e-a.htm
Comment 1 Alexey Proskuryakov 2006-02-15 12:44:17 PST
I think that the reason why you are seeing failures is that you don't have Ahem font installed (<http://webkit.opendarwin.org/quality/Ahem.ttf>). 

This special simple font is used to construct certain types of predictable tests. A page on Ian Hickson's site <http://www.hixie.ch/resources/fonts/> has a more detailed explanation.
Comment 2 Diego Miguel Virasoro 2006-02-15 13:16:42 PST
Indeed you were right. That has chanaged the appearances a lot. However there are still some small problems.

http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5506-ipadn-t-00-b-a.htm
1px horizontal link on top of the green bar

http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5506-ipadn-t-01-b-a.htm
lighter green line across the bar (exactly halfway)

http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5506-padn-t-00-b-a.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5508-ipadn-b-03-b-a.htm
Both show a small white column (maybe width=7px) between the colorful rows and the black right border

http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5507-ipadn-r-02-b-ag.htm
lighter green vertical line across the square (exactly halfway)

http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5508-ipadn-b-00-b-a.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5508-ipadn-b-01-f-a.htm
both show the lighter green vertical line across the bar (exactly halway)

http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5509-ipadn-l-02-b-ag.htm
there is also a red square which should be absent
Comment 3 David Kilzer (:ddkilzer) 2006-02-15 13:28:21 PST
I think the reporter would like this bug to be reopened per Comment #2.
Comment 4 Alexey Proskuryakov 2006-02-15 13:33:03 PST
Three of these tests are failing for me, and have identically broken results checked in:
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5509-ipadn-l-02-b-ag.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5508-ipadn-b-03-b-a.htm
http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5506-padn-t-00-b-a.htm

Other tests start to break if I zoom in with Cmd+=. This might be an expected behavior.
Comment 5 Gustaaf Groenendaal (MysteryQuest) 2006-02-15 15:00:37 PST
Confirming this, but please note that CSS 2.1 is not fully supported and that this is a to-do.

Description: Add padding to CSS 2.1
Comment 6 Joost de Valk (AlthA) 2006-02-15 15:05:18 PST
This is still under discussion, however, i HAVE found this problem to be real myself as well.
Comment 7 Robert Blaut 2008-02-05 13:14:30 PST
All above mentioned tests renders correctly (except this test: http://www.w3.org/Style/CSS/Test/CSS2.1/current/t0804-c5509-ipadn-l-02-b-ag.htm) in Webkit r30001.

Other issues are related to antialiasing problem described in bug 17806. After turning font antialiasing off the issues disappear.
Comment 8 Robert Blaut 2008-02-05 13:17:14 PST
I mean bug 17086 :o
Comment 9 Anatoli Papirovski 2008-05-08 07:44:36 PDT
Created attachment 21016 [details]
patch

I've created a patch for this, however the bidi.cpp is rather confusing, so I'm just hoping I did not break anything else... I ran all the tests from LayoutTests and nothing seems to be broken.

I'm happy to change whatever, since as I said, I barely knew what I was doing...
Comment 10 mitz 2008-05-08 08:33:58 PDT
Created attachment 21018 [details]
A test case without padding

I don't understand:
1) Why the test is correct
2) How the failure is specific to padding (and therefore why the patch is correct)

Is WebKit's rendering of this test case correct? If not, I think that's what should be fixed, not anything specific to padding.
Comment 11 Robert Blaut 2008-05-08 23:04:04 PDT
Created attachment 21032 [details]
test case for "spanned" string wrapping 

I think the initial CSS 2.1 test exposes bug in Webkit which is completely unrelated to padding.

I my opinion Webkit incorrectly wraps "spanned" strings: "This_line<span>should_not</span>be_wrapped."
Comment 12 mitz 2008-05-09 18:02:32 PDT
Comment on attachment 21016 [details]
patch

This does not seem to address the root cause of the test failure.
Comment 13 Anatoli Papirovski 2008-05-09 19:03:06 PDT
Hmm, makes sense to me... For some reason I completely missed the fact that it was x<span>x</span> rathern than a padding issue... 
Comment 14 mitz 2008-05-25 19:49:52 PDT
Bug 18926 is essentially the same issue.
Comment 15 Ahmad Saleem 2022-06-09 17:40:15 PDT
Based on attached test cases, I am unable to reproduce the bug in Safari 15.5 on macOS 12.4 from visual perspective, there is no difference between Safari 15.5 with other browsers (Chrome Canary 104 and Firefox Nightly 103).

Is it something, which got fixed along the lines? Thanks!
Comment 16 zalan 2022-06-09 21:46:06 PDT
(In reply to Ahmad Saleem from comment #15)
> Based on attached test cases, I am unable to reproduce the bug in Safari
> 15.5 on macOS 12.4 from visual perspective, there is no difference between
> Safari 15.5 with other browsers (Chrome Canary 104 and Firefox Nightly 103).
> 
> Is it something, which got fixed along the lines? Thanks!
Yeah I am pretty sure it did. This is rather basic.