516288 - PresShell::GoToAnchor ignore scroll position changed by user

Status: NEW

(Core :: Layout, defect)

Description

[Oleg Romashin (:romaxa)]

Problem is easily reproducible on slow device. (Reproducible on nightly Fennec)
Open URL.
Click on smal image thumbnail.
When new page start loading, and some content is already visible, scroll page down.

EXP:
Page scrolled and stay on the same position

ACT:
Page scroll position moved back to top.


Backtrace:
#0  nsGfxScrollFrameInner::ScrollPositionWillChange (this=0x133167c, aScrollable=0xf8368c, aX=0, aY=5400)
    at layout/generic/nsGfxScrollFrame.cpp:1754
#1  0x40f6f4f8 in nsScrollPortView::ScrollToImpl (this=0xf83638, aX=0, aY=5400)
    at view/src/nsScrollPortView.cpp:691
#2  0x40f6fc88 in nsScrollPortView::ScrollTo (this=0xf83638, aDestinationX=<value optimized out>, aDestinationY=<value optimized out>, 
    aUpdateFlags=<value optimized out>)
    at view/src/nsScrollPortView.cpp:255
#3  0x40cada14 in PresShell::DoScrollContentIntoView (this=0xf8368c, aContent=<value optimized out>, aVPercent=) at layout/base/nsPresShell.cpp:4299
#4  0x40cb2a18 in PresShell::FlushPendingNotifications (this=0xccd820, aType=Flush_Layout)
    at layout/base/nsPresShell.cpp:4927
#5  0x40e3b554 in nsDocument::FlushPendingNotifications (this=<value optimized out>, aType=16266892)
    at content/base/src/nsDocument.cpp:6266
#6  0x411a7bec in nsDocShell::GetPositionAndSize (this=0xeee070, x=0xbef42b94, y=0xbef42b90, cx=0x0, cy=0x0)
    at docshell/base/nsDocShell.cpp:4378
#7  0x40cdee7c in nsSubDocumentFrame::ReflowFinished (this=0xdd9198)
    at layout/generic/nsFrameFrame.cpp:613
#8  0x40ca9f14 in PresShell::HandlePostedReflowCallbacks (this=0xccd820, aInterruptible=1)
    at layout/base/nsPresShell.cpp:4815
#9  0x40caa534 in PresShell::DidDoReflow (this=0xccd820, aInterruptible=1)
    at layout/base/nsPresShell.cpp:7079
#10 0x40cb27d0 in PresShell::ProcessReflowCommands (this=0xccd820, aInterruptible=1)
    at layout/base/nsPresShell.cpp:7331
#11 0x40cb29f4 in PresShell::FlushPendingNotifications (this=0xccd820, aType=Flush_InterruptibleLayout)
    at layout/base/nsPresShell.cpp:4924
#12 0x40e3b554 in nsDocument::FlushPendingNotifications (this=<value optimized out>, aType=16266892)
    at content/base/src/nsDocument.cpp:6266
#13 0x40cade04 in PresShell::ScrollContentIntoView (this=0xccd820, aContent=0x5a2470, aVPercent=0, aHPercent=-1)
    at layout/base/nsPresShell.cpp:4317
#14 0x40cb5fc4 in PresShell::GoToAnchor (this=0xccd820, aAnchorName=@0xbef4302c, aScroll=1)
    at layout/base/nsPresShell.cpp:3993
#15 0x40e0fe50 in nsContentSink::ScrollToRef (this=0xb088c0)
    at content/base/src/nsContentSink.cpp:1204
#16 0x40f16d44 in HTMLContentSink::DidBuildModel (this=<value optimized out>)
    at content/html/document/src/nsHTMLContentSink.cpp:1817
#17 0x40c239e0 in CNavDTD::DidBuildModel (this=0xea1560, anErrorCode=16266892, aNotifySink=0, aParser=<value optimized out>, aSink=0xb08970)
    at parser/htmlparser/src/CNavDTD.cpp:462
#18 0x40c2e1a4 in nsParser::DidBuildModel (this=0xe41b60, anErrorCode=0)
    at parser/htmlparser/src/nsParser.cpp:1563
#19 0x40c327f4 in nsParser::ResumeParse (this=0xe41b60, allowIteration=1, aIsFinalChunk=1, aCanInterrupt=1)
    at parser/htmlparser/src/nsParser.cpp:2312
#20 0x40c30320 in nsParser::ContinueInterruptedParsing (this=0xe41b60)
    at parser/htmlparser/src/nsParser.cpp:1771
#21 0x40e0e5f8 in nsContentSink::ContinueInterruptedParsingIfEnabled (this=0xb088c0)
    at content/base/src/nsContentSink.cpp:1726
#22 0x40e12724 in nsRunnableMethod<nsContentSink, void>::Run (this=<value optimized out>) at ../../../dist/include/nsThreadUtils.h:264
#23 0x413cfebc in nsThread::ProcessNextEvent (this=0x5b648, mayWait=0, result=0xbef4324c)
    at xpcom/threads/nsThread.cpp:516
#24 0x41397980 in NS_ProcessPendingEvents_P (thread=0x5b648, timeout=20) at nsThreadUtils.cpp:180
#25 0x4135b854 in nsBaseAppShell::NativeEventCallback (this=0x44d0d8)
    at widget/src/xpwidgets/nsBaseAppShell.cpp:121
#26 0x413520a4 in nsAppShell::EventProcessorCallback (source=<value optimized out>, condition=<value optimized out>, data=<value optimized out>)
    at widget/src/gtk2/nsAppShell.cpp:70

Comment 1

[Boris Zbarsky [:bzbarsky]]

Pretty sure we have existing bugs covering this.