Bug 104458

Summary: [EFL] [WebGL] Path is not resized correctly
Product: WebKit Reporter: Kalyan <kalyan.kondapally>
Component: WebKit EFLAssignee: Kalyan <kalyan.kondapally>
Status: RESOLVED FIXED    
Severity: Normal CC: dino, kenneth, lucas.de.marchi, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Bug Depends on:    
Bug Blocks: 103105, 104459    
Attachments:
Description Flags
patch
kenneth: review+, kenneth: commit-queue-
patchv2 none

Description Kalyan 2012-12-08 15:37:51 PST
In GraphicsContext3D::paintToCanvas , we flip the contents and draw on the surface.  The operations needed to flip the image(translation and scale ) are done before the rectangle with correct size is added to current path. This results in drawing pixels at wrong position. 

We could also combine the translation and scale operations into one by using matrix.
Comment 1 Kalyan 2012-12-08 15:41:31 PST
(In reply to comment #0)
> In GraphicsContext3D::paintToCanvas , we flip the contents and draw on the surface.  The operations needed to flip the image(translation and scale ) are done before the rectangle with correct size is added to current path. This results in drawing pixels at wrong position. 
> 
> We could also combine the translation and scale operations into one by using matrix.

Problem can be seen with LayoutTests/fast/canvas/webgl/canvas-test.html
Comment 2 Kalyan 2012-12-08 16:10:29 PST
Created attachment 178383 [details]
patch
Comment 3 Kenneth Rohde Christiansen 2012-12-09 06:34:27 PST
Comment on attachment 178383 [details]
patch

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

> Source/WebCore/ChangeLog:9
> +        The operations done to flip the image (translation and scale ) are done before rectangle with correct size

remove space before )

> Source/WebCore/ChangeLog:10
> +        is added to current path. This results in updating pixels at wrong positions. .

Remove the trailing " ."
Comment 4 Kalyan 2012-12-09 13:59:35 PST
Created attachment 178436 [details]
patchv2

review changes
Comment 5 WebKit Review Bot 2012-12-09 14:57:49 PST
Comment on attachment 178436 [details]
patchv2

Clearing flags on attachment: 178436

Committed r137098: <http://trac.webkit.org/changeset/137098>
Comment 6 WebKit Review Bot 2012-12-09 14:57:53 PST
All reviewed patches have been landed.  Closing bug.