[Webkit-unassigned] [Bug 32419] Rewrite CSS namespaces layout tests with dumpAsText

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=32419


Maciej Stachowiak <mjs at apple.com> changed:

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




--- Comment #8 from Maciej Stachowiak <mjs at apple.com>  2009-12-29 07:57:51 PST ---
(From update of attachment 44678)
I don't think the new tests are equivalent to the old ones. Here are some
differences:

1) The original tests have the relevant styles inline, not in an external
stylesheets included via ?xml-stylesheet PI.
2) The original tests use elements that are present as markup in the document
as the test subject.
3) Some of the tests had a non-HTML root element and in general a different
tree structure than what your test creates.

All of these changes alter the code paths tested by the tests.

Furthermore, these tests are imported from the official CSS Namespaces test
suite. I don't think we should be making such substantial changes to tests from
an external test suite. I think it would be ok to add the new text-dumping
tests but I do not think the old ones should be removed.

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