[webkit-reviews] review requested: [Bug 74680] [Plugins GTK/QT] Signed to Unsigned Conversion Error : [Attachment 135250] Signed to Unsigned Conversion Error
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Apr 2 18:08:14 PDT 2012
Kwangmin Bang <justine.bang at samsung.com> has asked for review:
Bug 74680: [Plugins GTK/QT] Signed to Unsigned Conversion Error
https://bugs.webkit.org/show_bug.cgi?id=74680
Attachment 135250: Signed to Unsigned Conversion Error
https://bugs.webkit.org/attachment.cgi?id=135250&action=review
------- Additional Comments from Kwangmin Bang <justine.bang at samsung.com>
rebase the patch.
When tested webkit for desktop, it's not need this patch.
My testing environment is using floating viewport for mobile browser( view is
bigger than device screen).
For getting m_windowRect, I'm using contentsToScreen instead of
contentsToWindow.
so, m_windowRect.x or y can be the negative.
However, I think these codes are a basic problem about conversion. ( unsigned =
signed )
this patch will prevent "Signed to Unsigned Conversion Error" and remove the
warning in build time.
How do you think about my opinion ?
More information about the webkit-reviews
mailing list