RESOLVED FIXED 41974
new baseline for bidi-mirror-he-ar.html and bidi-neutral-run.html (after r62965) 
https://bugs.webkit.org/show_bug.cgi?id=41974
Summary new baseline for bidi-mirror-he-ar.html and bidi-neutral-run.html (after r62965)
Xiaomei Ji
Reported 2010-07-09 12:17:29 PDT
platform/mac saves expected layout test results from SnowLeopard. platform/mac-leopard saves expected layout test results from Leopard.
Attachments
path /w layout test (5.25 KB, patch)
2010-07-09 12:32 PDT, Xiaomei Ji 		no flags
DetailsFormatted DiffDiff
path /w layout test (75.07 KB, patch)
2010-07-09 13:11 PDT, Xiaomei Ji 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Xiaomei Ji
Comment 1 2010-07-09 12:25:30 PDT
And they need rebaseline in Chromium.
Xiaomei Ji
Comment 2 2010-07-09 12:32:33 PDT
Created attachment 61075 [details] path /w layout test
Xiaomei Ji
Comment 3 2010-07-09 12:37:24 PDT
Committed r62979: <http://trac.webkit.org/changeset/62979>
Xiaomei Ji
Comment 4 2010-07-09 13:11:19 PDT
Created attachment 61078 [details] path /w layout test
Xiaomei Ji
Comment 5 2010-07-09 13:11:55 PDT
Comment on attachment 61075 [details] path /w layout test clear r+ on the commited patch
Xiaomei Ji
Comment 6 2010-07-09 13:14:37 PDT
reopen for chromium rebaseline
David Levin
Comment 7 2010-07-09 13:17:32 PDT
Comment on attachment 61078 [details] path /w layout test PLease address the following items and check it in. Thanks, Dave LayoutTests/platform/chromium/test_expectations.txt:3057 + // Newly added from r62965 by xji@chromium.org. Probably needs rebaseline. Pls remove this line. LayoutTests/ChangeLog:6 + Rebaseline bidi-mirror-he-ar.html and bidi-neutral-run.html for Chromium after r62965 Lots of spaces after Chromium. LayoutTests/ChangeLog:5 + Need a short description and bug URL (OOPS!) This line shouldn't be here.
Xiaomei Ji
Comment 8 2010-07-09 13:33:16 PDT
Committed r62988: <http://trac.webkit.org/changeset/62988>
Xiaomei Ji
Comment 9 2010-07-09 13:34:49 PDT
Comment on attachment 61078 [details] path /w layout test clear the r+ on committed patch
Note You need to log in before you can comment on or make changes to this bug.