Changeset 133106 in webkit

Timestamp:

Oct 31, 2012 6:21:26 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[CSS Exclusions] Test that dynamically setting shape-inside causes relayout in child elements
​https://bugs.webkit.org/show_bug.cgi?id=97721

Patch by Bear Travis <​betravis@adobe.com> on 2012-10-31
Reviewed by Dirk Schulze.

Adding a test that dynamically sets shape-inside on a block containing multiple block children.

• fast/exclusions/resources/simple-rectangle.js:

(createRectangleTest): Modifying test to work with elements with existing content.
(createRectangleTestResult): Modifying test expectation to work with elements with existing content.

• fast/exclusions/shape-inside/shape-inside-multiple-blocks-dynamic-expected.html: Added.
• fast/exclusions/shape-inside/shape-inside-multiple-blocks-dynamic.html: Added.

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• fast/exclusions/resources/simple-rectangle.js (modified) (2 diffs)
• fast/exclusions/shape-inside/shape-inside-multiple-blocks-dynamic-expected.html (added)
• fast/exclusions/shape-inside/shape-inside-multiple-blocks-dynamic.html (added)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2012-10-31  Bear Travis  <betravis@adobe.com>
+
+        [CSS Exclusions] Test that dynamically setting shape-inside causes relayout in child elements
+        https://bugs.webkit.org/show_bug.cgi?id=97721
+
+        Reviewed by Dirk Schulze.
+
+        Adding a test that dynamically sets shape-inside on a block containing multiple block children.
+
+        * fast/exclusions/resources/simple-rectangle.js:
+        (createRectangleTest): Modifying test to work with elements with existing content.
+        (createRectangleTestResult): Modifying test expectation to work with elements with existing content.
+        * fast/exclusions/shape-inside/shape-inside-multiple-blocks-dynamic-expected.html: Added.
+        * fast/exclusions/shape-inside/shape-inside-multiple-blocks-dynamic.html: Added.
+
 2012-10-31  Roger Fong  <roger_fong@apple.com>
 

trunk/LayoutTests/fast/exclusions/resources/simple-rectangle.js

@@ -26 +26 @@
     rules.push('border: 1px solid blue');
     stylesheet.insertRule('#' + elementId + '::before{' + rules.join(';') + '}');
-    elem.innerHTML = content;
+    if (content)
+        elem.innerHTML = content;
 }
 
@@ -52 +53 @@
     rules.push('border: 1px solid blue');
     stylesheet.insertRule('#' + elementId + '::before{' + rules.join(';') + '}');
-    elem.innerHTML = content;
+    if (content)
+        elem.innerHTML = content;
 }