[Webkit-unassigned] [Bug 63632] [CSS Regions] Add support for skipped tests

Wed Jun 29 16:09:08 PDT 2011

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

Tony Chang <tony at chromium.org> changed:

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

--- Comment #6 from Tony Chang <tony at chromium.org>  2011-06-29 16:09:08 PST ---
(From update of attachment 99163)
View in context: https://bugs.webkit.org/attachment.cgi?id=99163&action=review

Some minor nits to the test file.

> LayoutTests/fast/regions/flow-content-basic.html:1
> +<head>

<!DOCTYPE HTML>

> LayoutTests/fast/regions/flow-content-basic.html:6
> +    <!-- this test checks that content can be redirected to a region. It also
> +            tries out a validation strategy that recreates the intended display
> +            using non-region markup. If top does not match the bottom then the 
> +            result should be rejected -->

This indenting looks weird, maybe some tabs snuck in?  Also 'this test' -> 'This test' and end the last sentence with a period.

> LayoutTests/fast/regions/flow-content-basic.html:26
> +<body onload="runTests();">

This onload doesn't look used.

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