Commit 9ab464d1 authored by darin's avatar darin

Fix suggested by Rob Buis. Reviewed by Darin.

        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9341
          REGRESSION: Repro crash caused by style="font:bold"

        Test: fast/css/font-shorthand-weight-only.html

        * css/cssparser.cpp: (WebCore::CSSParser::parseFont): Revert part of the change
        made to fix bug 5564.



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@14761 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent f3820f23
cca04d743fec359f2efe44ad8bf896eb
\ No newline at end of file
layer at (0,0) size 800x600
RenderView at (0,0) size 800x600
layer at (0,0) size 800x600
RenderBlock {HTML} at (0,0) size 800x600
RenderBody {BODY} at (8,8) size 784x576
RenderBlock {P} at (0,0) size 784x18
RenderText {#text} at (0,0) size 53x18
text run at (0,0) width 53: "Test for "
RenderInline {I} at (0,0) size 716x18
RenderInline {A} at (0,0) size 348x18 [color=#0000EE]
RenderText {#text} at (53,0) size 348x18
text run at (53,0) width 348: "http://bugzilla.opendarwin.org/show_bug.cgi?id=9341"
RenderText {#text} at (401,0) size 368x18
text run at (401,0) width 4: " "
text run at (405,0) width 364: "REGRESSION: Repro crash caused by style=\"font:bold\""
RenderText {#text} at (769,0) size 4x18
text run at (769,0) width 4: "."
RenderBlock {P} at (0,34) size 784x18
RenderText {#text} at (0,0) size 127x18
text run at (0,0) width 127: "PASS (didn\x{2019}t crash)"
<html>
<head>
<title></title>
<style type="text/css">
br { font: bold; }
</style>
</head>
<body>
<p>
Test for <i><a href="http://bugzilla.opendarwin.org/show_bug.cgi?id=9341">http://bugzilla.opendarwin.org/show_bug.cgi?id=9341</a>
REGRESSION: Repro crash caused by style="font:bold"</i>.
</p>
<p>
PASS (didn&rsquo;t crash)
</p>
</body>
</html>
2006-06-07 Mitz Pettel <opendarwin.org@mitzpettel.com>
Fix suggested by Rob Buis. Reviewed by Darin.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9341
REGRESSION: Repro crash caused by style="font:bold"
Test: fast/css/font-shorthand-weight-only.html
* css/cssparser.cpp: (WebCore::CSSParser::parseFont): Revert part of the change
made to fix bug 5564.
2006-06-07 Mitz Pettel <opendarwin.org@mitzpettel.com>
Reviewed by Hyatt.
......
......@@ -2032,7 +2032,7 @@ bool CSSParser::parseFont(bool important)
break;
value = valueList->next();
}
if (strict && !value)
if (!value)
goto invalid;
// set undefined values to default
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment