Summary: | Switch paintOverflowControls to use IntPoint | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Levi Weintraub <leviw> | ||||||
Component: | Layout and Rendering | Assignee: | Levi Weintraub <leviw> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | darin, eae, eric, simon.fraser | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | 61883 | ||||||||
Bug Blocks: | 60318 | ||||||||
Attachments: |
|
Description
Levi Weintraub
2011-06-01 13:54:04 PDT
Created attachment 95666 [details]
Patch
Comment on attachment 95666 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=95666&action=review > Source/WebCore/rendering/RenderLayer.cpp:2284 > + m_cachedOverlayScrollbarOffset = paintOffset; eeewww. Wonder why this exists. > Source/WebCore/rendering/RenderLayer.cpp:2295 > + adjustedPaintOffset.move(m_cachedOverlayScrollbarOffset); This doesn't look right. I think you meant =. > Source/WebCore/rendering/RenderLayerBacking.cpp:1105 > + int tx = layerBounds.x() - m_owningLayer->renderBoxX(); > + int ty = layerBounds.y() - m_owningLayer->renderBoxY(); I guess we're not ready to kill the tx, ty here yet? :) Comment on attachment 95666 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=95666&action=review Thanks for the catch! >> Source/WebCore/rendering/RenderLayer.cpp:2284 >> + m_cachedOverlayScrollbarOffset = paintOffset; > > eeewww. Wonder why this exists. I'll second that... Seems pretty funky to me! >> Source/WebCore/rendering/RenderLayer.cpp:2295 >> + adjustedPaintOffset.move(m_cachedOverlayScrollbarOffset); > > This doesn't look right. I think you meant =. D'oh! Good catch! >> Source/WebCore/rendering/RenderLayerBacking.cpp:1105 >> + int ty = layerBounds.y() - m_owningLayer->renderBoxY(); > > I guess we're not ready to kill the tx, ty here yet? :) paintLayer will unfortunately probably be one of the last, but we're making great progress! :) Created attachment 95668 [details]
Patch
Comment on attachment 95668 [details]
Patch
OK.
Comment on attachment 95668 [details] Patch Clearing flags on attachment: 95668 Committed r87864: <http://trac.webkit.org/changeset/87864> All reviewed patches have been landed. Closing bug. |