Bug 27584

Summary: [CSS3 Backgrounds and Borders] Handle the / and 4 values in border-radius
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: CSSAssignee: mitz
Status: RESOLVED FIXED    
Severity: Normal CC: bugzilla33, mitz
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 27569    
Attachments:
Description Flags
Support the new border-radius syntax and maintain legacy -webkit-border-radius syntax simon.fraser: review+

Simon Fraser (smfr)
Reported 2009-07-22 16:11:00 PDT
CSS 3 border radius has the ability to specify 4 values at one time, and separate values with a /. We need to handle these. We already handle the radius-reducing algorithm in the spec.
Attachments
Support the new border-radius syntax and maintain legacy -webkit-border-radius syntax (36.94 KB, patch)
2009-07-28 17:02 PDT, mitz 		simon.fraser: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2009-07-28 17:02:36 PDT
Created attachment 33680 [details] Support the new border-radius syntax and maintain legacy -webkit-border-radius syntax
Simon Fraser (smfr)
Comment 2 2009-07-28 17:13:43 PDT
Comment on attachment 33680 [details] Support the new border-radius syntax and maintain legacy -webkit-border-radius syntax > Index: WebCore/css/CSSParser.cpp > =================================================================== > + case CSSPropertyBorderRadius: > + case CSSPropertyWebkitBorderRadius: > + return parseBorderRadius(propId, important); > + break; No need for the break after return. r=me
mitz
Comment 3 2009-07-28 17:17:45 PDT
Fixed in <http://trac.webkit.org/changeset/46508>.
Adam Roben (:aroben)
Comment 4 2009-07-28 17:18:26 PDT
*** Bug 23675 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.