[webkit-reviews] review granted: [Bug 42272] Canvas: Fast-path for assigning the same color string as before to fillStyle or strokeStyle : [Attachment 61546] Proposed patch v2

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jul 14 12:19:24 PDT 2010


Darin Adler <darin at apple.com> has granted Andreas Kling
<andreas.kling at nokia.com>'s request for review:
Bug 42272: Canvas: Fast-path for assigning the same color string as before to
fillStyle or strokeStyle
https://bugs.webkit.org/show_bug.cgi?id=42272

Attachment 61546: Proposed patch v2
https://bugs.webkit.org/attachment.cgi?id=61546&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
>  JSValue JSCanvasRenderingContext2D::strokeStyle(ExecState* exec) const
>  {
>      CanvasRenderingContext2D* context =
static_cast<CanvasRenderingContext2D*>(impl());
> +
>      return toJS(exec, context->strokeStyle());	 
>  }

Stray change here.


More information about the webkit-reviews mailing list