[Webkit-unassigned] [Bug 23262] Dual lines in css2.1 layout tests do not match:

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Sep 3 00:32:54 PDT 2009


Eric Seidel <eric at webkit.org> changed:

           What    |Removed                     |Added
  Attachment #35109|review?                     |review-
               Flag|                            |

--- Comment #5 from Eric Seidel <eric at webkit.org>  2009-09-03 00:32:54 PDT ---
(From update of attachment 35109)
The experts have been silent about your patch for over 2 weeks.  I'm not an
expert here, but your patch looks non-harmful.

Some nits:

Why the BOM?

Why all the extra junk in the tests?
 5   <link rel="help" href="http://www.w3.org/TR/CSS21/generate.html#counters">
 6   <link rel="help"
 7   <link rel="help" href="http://www.w3.org/TR/CSS21/syndata.html#counter">

No need to run this off a timeout:
 37  <body onload="setTimeout('run()', 0)">

Better to just run it from inline script by placing the script tag low enough
in the body...  Then you don't need "notifyDone()" either.

Do these tests really need to dump the render tree?  Ideally tests should be
js-only test first, if they can't be js-only, at least dump as text, and render
tree dumping tests should be used as a last-resort.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list