Closed Bug 293221 Opened 15 years ago Closed 15 years ago

CanvasRenderingContext2D has wrong default property values

Categories

(Core :: Canvas: 2D, defect)

x86
Windows 2000
defect
Not set

Tracking

()

VERIFIED FIXED

People

(Reporter: jens.b, Unassigned)

Details

(Keywords: testcase)

Attachments

(1 file)

Some of CanvasRenderingContext2D's properties have wrong values according to the
spec at http://whatwg.org/specs/web-apps/current-work/#canvas . One property,
lineCap, even throws an exception upon reading.
Note: The test case includes all properties, not only the ones with wrong values.
Keywords: testcase
Fixed with patch in bug 294400 - note that one of the values (the composite
operator) will still be wrong, because the cairo names are a little strange and
not inline with canvas/quartz usage.  That gets fixed once we move to a more
recent version of cairo.

Another one of these values has the correct default, it's just incorrect in the
spec :)
fixed by checkin for bug 29440
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Which one is incorrect in the specification? Would be nice if that specification
was fixed so we actually have clue against what we can develop test cases and such.
Vlad and I had a little talk. Nothing to see here. Move along now.
(In reply to comment #5)
> Which one is incorrect in the specification? Would be nice if that specification
> was fixed so we actually have clue against what we can develop test cases and
such.

Sorry, is a bug in the testcase ;)  The default composite op specified by the
spec is "source-over", not "over".

There's a discrepancy between our impl and safari (they have "source-over", we
have "over" as the default, but they behave the same) -- cairo names these
things very differently at the moment, and that won't be fixed until we move to
a CVS version of cairo.  Once that happens we'll nail down exactly which
operators correspond to which. (I haven't spent time doing that up to now, since
we'll be moving to a new cairo as soon as 1.8b2 goes out the door.)
Thanks.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.