[Webkit-unassigned] [Bug 20435] Canvas missing exceptioncode for gradients

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 20 13:06:27 PDT 2008


oliver at apple.com changed:

           What    |Removed                     |Added
  Attachment #22897|review?                     |review-
               Flag|                            |

------- Comment #7 from oliver at apple.com  2008-08-20 13:06 PDT -------
(From update of attachment 22897)
fixing draw image should go in a separate bug, however issues i spotted, that
make me r- this in its current form:
You changed FloatRect::contains -- i'm not entirely happy with this as
FloatRect as the contains semantics would be subtly different from
IntRect::contains.  A better approach would to normalise the rect before doing
the contains check.

The code also now ends up checking for !srcRect.width(), etc multiple times
throwing an exception in one case and silently returning in another.

Your testcase also isn't particularly great as you  don't actually test that
you can specify a negative sized source rect (0-width/height for some of the
components result in an exception), and you don't test the rendering behaviour
of a negative source.  This last part is especially critical as you are adding
new functionality, so ou need to confirm that the new functionality actually

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

More information about the webkit-unassigned mailing list