Title: [146012] branches/chromium/1441/Source/WebCore/rendering/RenderLayer.cpp
Revision
146012
Author
[email protected]
Date
2013-03-17 10:12:19 -0700 (Sun, 17 Mar 2013)

Log Message

Revert 143825 "RenderLayer::scrollTo() should call FrameLoaderCl..."

> RenderLayer::scrollTo() should call FrameLoaderClient::didChangeScrollOffset()
> https://bugs.webkit.org/show_bug.cgi?id=110673
> -and corresponding-
> <rdar://problem/13258596>
> 
> Reviewed by Sam Weinig.
> 
> FrameLoaderClient::didChangeScrollOffset() doesn't get called for web pages that
> have overflow on the body. We can easily address this by calling it at the 
> end of RenderLayer::scrollTo().
> 
> * rendering/RenderLayer.cpp:
> (WebCore::RenderLayer::scrollTo):
> 
> 

[email protected]
Review URL: https://codereview.chromium.org/12851007

Modified Paths

Diff

Modified: branches/chromium/1441/Source/WebCore/rendering/RenderLayer.cpp (146011 => 146012)


--- branches/chromium/1441/Source/WebCore/rendering/RenderLayer.cpp	2013-03-17 16:49:43 UTC (rev 146011)
+++ branches/chromium/1441/Source/WebCore/rendering/RenderLayer.cpp	2013-03-17 17:12:19 UTC (rev 146012)
@@ -2214,7 +2214,6 @@
         renderer()->node()->document()->eventQueue()->enqueueOrDispatchScrollEvent(renderer()->node(), DocumentEventQueue::ScrollEventElementTarget);
 
     InspectorInstrumentation::didScrollLayer(frame);
-    frame->loader()->client()->didChangeScrollOffset();
 }
 
 static inline bool frameElementAndViewPermitScroll(HTMLFrameElement* frameElement, FrameView* frameView) 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to