WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
42803
Convert DRT list tests from pixel-tests/render tree dumps to text
https://bugs.webkit.org/show_bug.cgi?id=42803
Summary
Convert DRT list tests from pixel-tests/render tree dumps to text
Daniel Bates
Reported
2010-07-21 20:54:29 PDT
With the passing of
Bug #37929
and
Bug #37930
, we can convert a subset of the DRT lists tests from pixel-tests/render tree dumps to text.
Attachments
[Patch] Part 1 of 2 (Updated tests and expected results)
(30.05 KB, patch)
2010-07-21 21:07 PDT
,
Daniel Bates
no flags
Details
Formatted Diff
Diff
[Patch] Part 2 of 2 (Remove Mac and Windows render-tree/pixel-tests results)
(814.25 KB, patch)
2010-07-21 21:10 PDT
,
Daniel Bates
darin
: review+
Details
Formatted Diff
Diff
[Patch] Part 1 of 2 (Updated tests and expected results)
(30.08 KB, patch)
2010-07-21 21:21 PDT
,
Daniel Bates
no flags
Details
Formatted Diff
Diff
[Patch] Part 1 of 2 (Updated tests and expected results)
(30.32 KB, patch)
2010-07-21 21:45 PDT
,
Daniel Bates
darin
: review+
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Daniel Bates
Comment 1
2010-07-21 21:07:03 PDT
Created
attachment 62257
[details]
[Patch] Part 1 of 2 (Updated tests and expected results)
Daniel Bates
Comment 2
2010-07-21 21:10:11 PDT
Created
attachment 62258
[details]
[Patch] Part 2 of 2 (Remove Mac and Windows render-tree/pixel-tests results) Remove the render-tree/pixel-test results for some of the Mac and Windows DRT list tests since we are converting these tests to text.
WebKit Review Bot
Comment 3
2010-07-21 21:14:17 PDT
Attachment 62258
[details]
did not pass style-queue: Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1 Total errors found: 0 in 0 files If any of these errors are false positives, please file a bug against check-webkit-style.
Daniel Bates
Comment 4
2010-07-21 21:18:08 PDT
Since the patch is fairly large, I thought to break it up into two parts to make it easier to review the changes. Let me know [Patch] Part 1 of 2 (Updated tests and expected results): Includes the modifications of the test files so that they dump text. Also includes the text-based expected results and change log entry. [Patch] Part 2 of 2 (Remove Mac and Windows render-tree/pixel-tests results): The patch removes all of the render-tree/pixel-test results of the converted tests. I converted the following tests to text: fast/lists/alpha-boundary-values.html fast/lists/alpha-list-wrap.html fast/lists/decimal-leading-zero.html fast/lists/li-values.html fast/lists/list-style-type-dynamic-change.html fast/lists/w3-css3-list-styles-alphabetic.html fast/lists/w3-css3-list-styles-numeric.html fast/lists/w3-list-styles.html Let me know if I should break up either part of this patch further.
Daniel Bates
Comment 5
2010-07-21 21:21:59 PDT
Created
attachment 62259
[details]
[Patch] Part 1 of 2 (Updated tests and expected results) Update change log so that the line for "fast/lists/list-style-type-dynamic-change.html" has the remark "Modified to dump as text."
Daniel Bates
Comment 6
2010-07-21 21:45:59 PDT
Created
attachment 62261
[details]
[Patch] Part 1 of 2 (Updated tests and expected results) Fix typos in fast/lists/w3-css3-list-styles-numeric.html for binary list-style-type results.
Darin Adler
Comment 7
2010-07-24 12:48:16 PDT
Comment on
attachment 62261
[details]
[Patch] Part 1 of 2 (Updated tests and expected results) These new versions of the tests all seem OK and some even seem great.
Daniel Bates
Comment 8
2010-07-27 20:45:28 PDT
Committed
r64186
: <
http://trac.webkit.org/changeset/64186
>
WebKit Review Bot
Comment 9
2010-07-27 21:01:33 PDT
http://trac.webkit.org/changeset/64186
might have broken Qt Linux Release
Daniel Bates
Comment 10
2010-07-28 00:15:56 PDT
(In reply to
comment #9
)
>
http://trac.webkit.org/changeset/64186
might have broken Qt Linux Release
Inadvertently did not remove the platform-specific expected results in the Qt and GTK ports. These platform-specific render-tree/pixel test results are no longer needed since the test dump as text. Removed these files in changeset 64188 <
http://trac.webkit.org/changeset/64188
>.
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