[Webkit-unassigned] [Bug 38651] WebFrame::paintDocumentRectToContext paints content at the wrong location

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu May 6 10:04:24 PDT 2010


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


Adam Roben (aroben) <aroben at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #55252|review?                     |review+
               Flag|                            |




--- Comment #2 from Adam Roben (aroben) <aroben at apple.com>  2010-05-06 10:04:24 PST ---
(From update of attachment 55252)
> @@ -348,11 +348,8 @@ HRESULT STDMETHODCALLTYPE WebFrame::pain
>      gc.save();
>      LONG width = rect.right - rect.left;
>      LONG height = rect.bottom - rect.top;
> -    FloatRect dirtyRect;
> -    dirtyRect.setWidth(width);
> -    dirtyRect.setHeight(height);
> +    FloatRect dirtyRect(rect.left, rect.top, width, height);

You could get rid of the width and height variables by doing:

FloatRect dirtyRect(IntRect(rect));

Too bad we don't have a way to test this.

r=me

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



More information about the webkit-unassigned mailing list