[webkit-dev] currently failing regression tests on Mac OS X Leopard

Jungshik Shin (신정식, 申政湜) jshin at chromium.org
Fri Mar 6 12:04:07 PST 2009


Oops. I meant reply-to-all but just replied to Darin.

2009/3/6 Jungshik Shin (신정식, 申政湜) <jshin at chromium.org>

>
>
> 2009/3/6 Darin Adler <darin at apple.com>
>
>> On Mar 6, 2009, at 8:36 AM, Dimitri Glazkov wrote:
>>
>>  Since I checked in the failing tests, I feel responsible, too.
>>>
>>> I'd say let's remove these tests and work to get the CSS2.1 suite
>>> updated. WDYT?
>>>
>>
>> I think it's an acceptable strategy to have these new separate tests. I
>> just think someone needs to generate the correct expected results.
>>
>> Changing the CSS 2.1 suite itself would be OK, but I don't think it would
>> have a big effect. We'd still need correct expected results.
>
>
> Sorry for all the confusion. I mentioned the following on  IRC and in the
> bug, but perhaps, I was not clear enough.
>
> What happened  was that test results are correct, but actual test files
> (html files) that were svn-copied to a new location were NOT revised in the
> new place. (my patch has new html files, but during the landing, svn-apply
> did not work as intended and svn-copied files without actual changes went
> int).
> As a result, there's a mismatch between the expected result and what's
> supposed to be new test files (that ARE actually just the copies of the
> original tests).
>
> So, we can do any of the following (I'm also replying to Darin's question
> here).
>
> 1. What Dave suggested on IRC.
>
> 1a. Revise the original CSS 2.1 tests in place and remove what's added
> recently.
>    (this will be done by landing the latest patch in bug 23482)
>
> 1b. Same end result as 1a:
>        - roll out what's added recently right away without waiting for
> Dave's r+.
>        - once Dave gives me a nod for in-place modification of the original
> tests, make that change.
>
> 2. Leave the original tests alone and just update 'new' test html files in
> fast/block/float to match expected results.
>
> If we do 1a/1b,  I'm willing to talk to Ian once more about our changes and
> talk him into revising CSS 2.1 suite the same way. I already asked him in
> the bugzilla, but he may have missed it.
>
> #2 was what was agreed upon and what my second latest patch does, but
> yesterday, Dave said that it's better to revise the original tests in place.
> So, I made the latest patch (3rd patch in the bug) and asked him for
> review.
>
> Jungshik
>
>
>
>
>
>
>>
>>
>> I think the main question is who will generate expected results that are
>> correct for Mac OS X.
>>
>>    -- Darin
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20090306/6f113667/attachment.html>


More information about the webkit-dev mailing list