[Webkit-unassigned] [Bug 61528] Implement maxWidth check for canvas fill/strokeText

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=61528





--- Comment #36 from Philip Rogers <pdr at google.com>  2011-11-15 11:54:45 PST ---
(In reply to comment #35)
> (From update of attachment 115200 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=115200&action=review
> 
> > Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp:-2208
> > -        stateSaver.restore();
> 
> I see.  This is extra because the stateSaver RIIA would do it for us?

Yes. The reason for this is stateSaver.restore() is called in StateSaver's destructor which is called when stateSaver goes out of scope (this was pointed out by Andreas). This change just prevents the second, redundant restore call.

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