DevHeads.net

Review Request 108736: khtml/canvas: Ignore invalid values for globalAlpha instead of throwing an exception

Review request for kdelibs.

Description
khtml/canvas: Ignore invalid values for globalAlpha instead of throwing an exception

According to:
<a href="http://philip.html5.org/tests/canvas/suite/tests/spec.html#testrefs.2d.composite.globalAlpha.range" title="http://philip.html5.org/tests/canvas/suite/tests/spec.html#testrefs.2d.composite.globalAlpha.range">http://philip.html5.org/tests/canvas/suite/tests/spec.html#testrefs.2d.c...</a>
and test:
<a href="http://philip.html5.org/tests/canvas/suite/tests/2d.composite.globalAlpha.invalid.html" title="http://philip.html5.org/tests/canvas/suite/tests/2d.composite.globalAlpha.invalid.html">http://philip.html5.org/tests/canvas/suite/tests/2d.composite.globalAlph...</a>

Diffs
khtml/ecma/kjs_context2d.cpp de9870d

Diff: <a href="http://git.reviewboard.kde.org/r/108736/diff/" title="http://git.reviewboard.kde.org/r/108736/diff/">http://git.reviewboard.kde.org/r/108736/diff/</a>

Testing

Thanks,

Bernd Buschinski

Comments

Re: Review Request 108736: khtml/canvas: Ignore invalid values f

By Commit Hook at 03/30/2013 - 11:15

(Updated March 30, 2013, 3:15 p.m.)

Status
This change has been marked as submitted.

Review request for kdelibs.

Description
khtml/canvas: Ignore invalid values for globalAlpha instead of throwing an exception

According to:
<a href="http://philip.html5.org/tests/canvas/suite/tests/spec.html#testrefs.2d.composite.globalAlpha.range" title="http://philip.html5.org/tests/canvas/suite/tests/spec.html#testrefs.2d.composite.globalAlpha.range">http://philip.html5.org/tests/canvas/suite/tests/spec.html#testrefs.2d.c...</a>
and test:
<a href="http://philip.html5.org/tests/canvas/suite/tests/2d.composite.globalAlpha.invalid.html" title="http://philip.html5.org/tests/canvas/suite/tests/2d.composite.globalAlpha.invalid.html">http://philip.html5.org/tests/canvas/suite/tests/2d.composite.globalAlph...</a>

Diffs
khtml/ecma/kjs_context2d.cpp de9870d

Diff: <a href="http://git.reviewboard.kde.org/r/108736/diff/" title="http://git.reviewboard.kde.org/r/108736/diff/">http://git.reviewboard.kde.org/r/108736/diff/</a>

Testing

Thanks,

Bernd Buschinski

Re: Review Request 108736: khtml/canvas: Ignore invalid values f

By Commit Hook at 03/30/2013 - 11:13

This review has been submitted with commit 223c6c75e3c87bdfd3a9b263d419fa0baea905bb by Bernd Buschinski to branch master.

- Commit Hook

On Feb. 3, 2013, 4:41 p.m., Bernd Buschinski wrote: