Changeset 282442 in webkit


Ignore:
Timestamp:
Sep 15, 2021 12:19:07 AM (10 months ago)
Author:
mmaxfield@apple.com
Message:

CSSFontFaceSrcValue.cssText should be quoted consistently with other browsers
​https://bugs.webkit.org/show_bug.cgi?id=230273

Reviewed by Darin Adler.

Addressing post-review comments.

  • fast/css/font-face-src-parsing-expected.txt:
  • fast/css/font-face-src-parsing.html:
Location:
trunk/LayoutTests
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r282441 r282442  
     12021-09-15  Myles C. Maxfield  <mmaxfield@apple.com>
     2
     3        CSSFontFaceSrcValue.cssText should be quoted consistently with other browsers
     4        https://bugs.webkit.org/show_bug.cgi?id=230273
     5
     6        Reviewed by Darin Adler.
     7
     8        Addressing post-review comments.
     9
     10        * fast/css/font-face-src-parsing-expected.txt:
     11        * fast/css/font-face-src-parsing.html:
     12
    1132021-09-14  Simon Fraser  <simon.fraser@apple.com>
    214

  • trunk/LayoutTests/fast/css/font-face-src-parsing-expected.txt

    r282415 r282442  
    11Test parsing the src descriptor of @font-face.
    22
    3 Valid rules form the stylesheet:
     3Valid rules from the stylesheet:
    44
    55@font-face { src: url("font.ttf"); }
    … …  
    1515@font-face { src: local("foo"); }
    1616@font-face { src: local("foo bar"); }
    17 Expected result for valid rules:
    18 
    19 @font-face { src: url(font.ttf); }
    20 @font-face { src: url(http://foo/bar/font.ttf); }
    21 @font-face { src: url(font.svg#id); }
    22 @font-face { src: url(font.ttf) format(truetype); }
    23 @font-face { src: url(font.woff) format(woff), local(font2); }
    24 @font-face { src: url(font2.ttf); }
    25 @font-face { src: url(font2.ttf) format(truetype); }
    26 @font-face { src: url(font3.otf) format(opentype), local(foo bar); }
    27 @font-face { src: local(foo); }
    28 @font-face { src: local(font), local(foo bar); }
    29 @font-face { src: local(foo); }
    30 @font-face { src: local(foo bar); }
    31 Invalid rules form the stylesheet:
    32 
    33 @font-face { }
    34 @font-face { }
    35 @font-face { }
    36 @font-face { }
    37 @font-face { }
    38 @font-face { }
    39 @font-face { }
    40 @font-face { }
    41 Expected result for invalid rules:
     17Invalid rules from the stylesheet:
    4218
    4319@font-face { }

  • trunk/LayoutTests/fast/css/font-face-src-parsing.html

    r120683 r282442  
    6969<p>Test parsing the src descriptor of @font-face.</p>
    7070
    71 <p>Valid rules form the stylesheet:</p>
     71<p>Valid rules from the stylesheet:</p>
    7272<div id="validResult"></div>
    7373
    74 <p>Expected result for valid rules:</p>
    75 <pre id="validExpected">@font-face { src: url(font.ttf); }
    76 @font-face { src: url(http://foo/bar/font.ttf); }
    77 @font-face { src: url(font.svg#id); }
    78 @font-face { src: url(font.ttf) format(truetype); }
    79 @font-face { src: url(font.woff) format(woff), local(font2); }
    80 @font-face { src: url(font2.ttf); }
    81 @font-face { src: url(font2.ttf) format(truetype); }
    82 @font-face { src: url(font3.otf) format(opentype), local(foo bar); }
    83 @font-face { src: local(foo); }
    84 @font-face { src: local(font), local(foo bar); }
    85 @font-face { src: local(foo); }
    86 @font-face { src: local(foo bar); }
    87 </pre>
    88 
    89 <p>Invalid rules form the stylesheet:</p>
     74<p>Invalid rules from the stylesheet:</p>
    9075<pre id="invalidResult"></pre>
    91 
    92 <p>Expected result for invalid rules:</p>
    93 <pre id="invalidExpected">@font-face { }
    94 @font-face { }
    95 @font-face { }
    96 @font-face { }
    97 @font-face { }
    98 @font-face { }
    99 @font-face { }
    100 @font-face { }
    101 </pre>
    10276
    10377<script>
Note: See TracChangeset for help on using the changeset viewer.