WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
58984
Replace bidi-004.html with its new version.
https://bugs.webkit.org/show_bug.cgi?id=58984
Summary
Replace bidi-004.html with its new version.
Yael
Reported
2011-04-20 06:39:15 PDT
The test bidi-004.html was removed yesterday from the CSS 2.1 test suite, as it was deemed wrong. WebKit should have the new version of the test, so that we don't get more comments like
https://bugs.webkit.org/show_bug.cgi?id=57459#c25
.
Attachments
Patch.
(113.75 KB, patch)
2011-04-20 06:44 PDT
,
Yael
no flags
Details
Formatted Diff
Diff
Patch.
(114.32 KB, patch)
2011-04-20 06:48 PDT
,
Yael
no flags
Details
Formatted Diff
Diff
Patch.
(113.75 KB, patch)
2011-04-20 07:00 PDT
,
Yael
eric
: review+
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Yael
Comment 1
2011-04-20 06:44:56 PDT
Created
attachment 90335
[details]
Patch.
Yael
Comment 2
2011-04-20 06:48:56 PDT
Created
attachment 90336
[details]
Patch. One more try :)
Yael
Comment 3
2011-04-20 07:00:18 PDT
Created
attachment 90337
[details]
Patch.
Yael
Comment 4
2011-04-20 07:19:21 PDT
Comment on
attachment 90337
[details]
Patch. Not sure why removing images is causing a problem. Will commit manually after review.
Eric Seidel (no email)
Comment 5
2011-04-20 07:58:36 PDT
Comment on
attachment 90337
[details]
Patch. Your local checkout is not correctly attributing pngs as binary.
Eric Seidel (no email)
Comment 6
2011-04-20 07:59:12 PDT
(At least I believe that's what happened to your diff.)
Eric Seidel (no email)
Comment 7
2011-04-20 08:06:16 PDT
I'm a bit confused with all this. Can you point me to the decision where bidi-04 was deemed wrong? Can you explain what your new bidi-04 does?
Yael
Comment 8
2011-04-20 08:21:30 PDT
(In reply to
comment #7
)
> I'm a bit confused with all this. Can you point me to the decision where bidi-04 was deemed wrong? Can you explain what your new bidi-04 does?
After a discussion on IRC last night, Hixie said he was going to send an e-mail to www-style about bidi-004.xht and bidi-004a.xht being wrong. I do not see that e-mail, so it probably did not happen yet. I created this bug based on Hixie's comment and the test he created in
http://www.hixie.ch/tests/adhoc/css/box/inline/bidi/012.html
.
Yael
Comment 9
2011-04-20 08:22:51 PDT
(In reply to
comment #5
)
> (From update of
attachment 90337
[details]
) > Your local checkout is not correctly attributing pngs as binary.
I set svn propset svn:mime-type on these files, but that did not help. Anything else I should do? thanks,
Levi Weintraub
Comment 10
2011-04-20 10:56:14 PDT
(In reply to
comment #7
)
> I'm a bit confused with all this. Can you point me to the decision where bidi-04 was deemed wrong? Can you explain what your new bidi-04 does?
Yael and I talked to Hixie on IRC and discussed the change I had made to treat BR's as paragraph separators that don't break out of style/dom directionality, but only unicode control characters. That change prevented us from passing the old bidi-004.html test, and after showing it to him, he concurred that in fact the test was flawed.
Eric Seidel (no email)
Comment 11
2011-04-20 11:05:59 PDT
Comment on
attachment 90337
[details]
Patch. It may not be your diff, it may be prettypatch which is causing the strageness.
Yael
Comment 12
2011-04-20 11:16:53 PDT
Committed
r84398
: <
http://trac.webkit.org/changeset/84398
>
WebKit Review Bot
Comment 13
2011-04-20 12:28:11 PDT
http://trac.webkit.org/changeset/84398
might have broken Windows 7 Release (Tests) The following tests are not passing: fast/borders/bidi-012.html
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug