Changeset 159344 in webkit


Ignore:
Timestamp:
Nov 15, 2013 10:06:11 AM (10 years ago)
Author:
zoltan@webkit.org
Message:

[CSS Shapes][CSS Regions] Simplify shape-inside-on-multiple-regions-with-negative-shape-top.html
​https://bugs.webkit.org/show_bug.cgi?id=123808

Reviewed by Mihnea Ovidenie.

Use a rectangle instead of a complex polygon in the test.

  • fast/regions/shape-inside/shape-inside-on-multiple-regions-with-negative-shape-top-expected.html:
  • fast/regions/shape-inside/shape-inside-on-multiple-regions-with-negative-shape-top.html:
Location:
trunk/LayoutTests
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r159341 r159344  
     12013-11-15  Zoltan Horvath  <zoltan@webkit.org>
     2
     3        [CSS Shapes][CSS Regions] Simplify shape-inside-on-multiple-regions-with-negative-shape-top.html
     4        https://bugs.webkit.org/show_bug.cgi?id=123808
     5
     6        Reviewed by Mihnea Ovidenie.
     7
     8        Use a rectangle instead of a complex polygon in the test.
     9
     10        * fast/regions/shape-inside/shape-inside-on-multiple-regions-with-negative-shape-top-expected.html:
     11        * fast/regions/shape-inside/shape-inside-on-multiple-regions-with-negative-shape-top.html:
     12
    1132013-11-15  Michał Pakuła vel Rutka  <m.pakula@samsung.com>
    214

  • trunk/LayoutTests/fast/regions/shape-inside/shape-inside-on-multiple-regions-with-negative-shape-top-expected.html

    r158219 r159344  
    33<head>
    44<style>
    5 article { font: 10px Ahem, sans-serif; }
     5article { font: 20px/1 Ahem, sans-serif; }
    66.regions {
    77    width: 260px;
     8    height: 200px;
    89    margin: 0px 20px 20px;
    9     padding: 7px;
    1010    background-color: #b0dcff;
    11     -webkit-shape-inside: polygon(20px -62px, 314px -62px, 314px 152px, 44px 152px);
    12     height: 200px;
     11    -webkit-shape-inside: rectangle(10px, -20px, 100%, 50%);
    1312}
    1413.overflow {
    15     margin-top: -34px;
     14    margin-top: -20px;
    1615    margin-left: 20px;
    17     padding: 7px;
    1816}
    1917</style>
    … …  
    2220<article>
    2321    <div class="regions">
    24         <p>
    25             CSS Pseudo-elements is a new proposal by Adobe to the W3C that allows multiple "before" and "after" pseudo-elements to be generated by a single element.
    26             In this example you can see how another region (the blue one) is added after the page is loaded and the CSSRegions object is
     22        X X X X X X X X X X X X X X X X X X X X X X X X
    2723    </div>
    2824    <div class="regions">
    29         created.
    30             <a href="http://example.org">This is a very long link!</a>
    31         </p>
    32         <p>
    33             Multiple pseudo-elements cover use cases collected on css-tricks.com.
    34         </p>
    35         <p>
    36            This is a prototype JavaScript implementation to give you a feel for how the code will look like and behave. Please keep in mind that
     25        X X X X X X X X X X X X X X X X X X X X X X X X
    3726    </div>
    3827    <div class="overflow">
    39         this is just... . . .
     28       X X X X X
    4029    </div>
    4130</article>
    42 <p style="margin-top: 30px">
    43 We have two regions, a polygon shape with negative top coordinates is applied on the regions. The content should flow from the first shape to the second shape. The overflow should
    44 be pushed down below the content box.
     31<p style="margin-top: 20px">
     32Requires Ahem font. We have two regions, a rectangle shape with negative top coordinate and 10px left offset is applied on the regions.
     33The content should flow from the first shape to the second shape. The overflow should be pushed down below the content box.
    4534</p>
    46 <p>Bug <a href="http://webkit.org/b/123103">123103</a>: [CSS Shapes][CSS Regions] Don't apply shape-inside when we have multiple auto-height regions and the height is not resolved</p>
     35<p>Bug <a href="http://webkit.org/b/123346">123346</a>: [CSS Shapes][CSS Regions] Don't apply shape-inside when we have multiple auto-height regions and the height is not resolved</p>
    4736</body>
    4837</html>

  • trunk/LayoutTests/fast/regions/shape-inside/shape-inside-on-multiple-regions-with-negative-shape-top.html

    r158219 r159344  
    44<style>
    55#source {
    6     font: 10px Ahem, sans-serif;
     6    font: 20px/1 Ahem, sans-serif;
    77    -webkit-flow-into: article;
    88}
    … …  
    1010    -webkit-flow-from: article;
    1111    width: 260px;
     12    height: 200px;
    1213    margin: 0px 20px 20px;
    13     padding: 7px;
    1414    background-color: #b0dcff;
    15     -webkit-shape-inside: polygon(20px -62px, 314px -62px, 314px 152px, 44px 152px);
    16     height: 200px;
     15    -webkit-shape-inside: rectangle(10px, -20px, 100%, 50%);
    1716}
    18 
    1917</style>
    2018</head>
    … …  
    2523</article>
    2624<div id="source">
    27     <p>
    28         CSS Pseudo-elements is a new proposal by Adobe to the W3C that allows multiple "before" and "after" pseudo-elements to be generated by a single element.
    29         In this example you can see how another region (the blue one) is added after the page is loaded and the CSSRegions object is created.
    30         <a href="http://example.org">This is a very long link!</a>
    31     </p>
    32     <p>
    33         Multiple pseudo-elements cover use cases collected on css-tricks.com.
    34     </p>
    35     <p>
    36         This is a prototype JavaScript implementation to give you a feel for how the code will look like and behave. Please keep in mind that this is just... . . .
    37     </p>
     25    X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X X
    3826</div>
    3927
    4028<p style="margin-top: 40px;">
    41 We have two regions, a polygon shape with negative top coordinates is applied on the regions. The content should flow from the first shape to the second shape. The overflow should
    42 be pushed down below the content box.
     29Requires Ahem font. We have two regions, a rectangle shape with negative top coordinate and 10px left offset is applied on the regions.
     30The content should flow from the first shape to the second shape. The overflow should be pushed down below the content box.
    4331</p>
    44 <p>Bug <a href="http://webkit.org/b/123103">123103</a>: [CSS Shapes][CSS Regions] Don't apply shape-inside when we have multiple auto-height regions and the height is not resolved</p>
     32<p>Bug <a href="http://webkit.org/b/123346">123346</a>: [CSS Shapes][CSS Regions] Don't apply shape-inside when we have multiple auto-height regions and the height is not resolved</p>
    4533</body>
    4634</html>
Note: See TracChangeset for help on using the changeset viewer.