Bug 72497

Summary:

GraphicsContextStateSaver's restore() called redundantly in CanvasRenderingContext2D

Product:

WebKit

Reporter:

Philip Rogers <pdr>

Component:

SVG

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Minor

CC:

darin, kling, webkit.review.bot, zimmermann

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Small patch to remove an extra restore() call.	none
Small patch to remove an extra restore() call.	kling: review+, kling: commit-queue-
Small patch to remove an extra restore() call.	none
Small patch to remove an extra restore() call.	none

Description Philip Rogers 2011-11-16 06:25:53 PST

Created attachment 115368 [details]
Small patch to remove an extra restore() call.

In https://bugs.webkit.org/show_bug.cgi?id=61528, it was pointed out (by Andreas Kling) that GraphicsContextStateSaver.restore() is called twice in CanvasRenderingContext2D::drawTextInternal.

Comment 1 Philip Rogers 2011-11-16 06:32:06 PST

Created attachment 115369 [details]
Small patch to remove an extra restore() call. 

Still had the old bugid. Updated 61528->72497

Comment 2 Andreas Kling 2011-11-16 08:38:02 PST

Comment on attachment 115369 [details]
Small patch to remove an extra restore() call. 

View in context: https://bugs.webkit.org/attachment.cgi?id=115369&action=review

> Source/WebCore/ChangeLog:8
> +        No new tests. (OOPS!)

Actually, cq will choke on this line, you need to remove it. :)

Comment 3 Philip Rogers 2011-11-16 08:41:20 PST

Created attachment 115386 [details]
Small patch to remove an extra restore() call. 

Removed line about no new tests.

Comment 4 Philip Rogers 2011-11-16 08:47:20 PST

Created attachment 115387 [details]
Small patch to remove an extra restore() call. 

Manual edit to patch file fail. Lets try this again..

Comment 5 WebKit Review Bot 2011-11-16 11:17:00 PST

Comment on attachment 115387 [details]
Small patch to remove an extra restore() call. 

Clearing flags on attachment: 115387

Committed r100475: <http://trac.webkit.org/changeset/100475>

Comment 6 WebKit Review Bot 2011-11-16 11:17:05 PST

All reviewed patches have been landed.  Closing bug.