654741 - Notify Fennec chrome of window.scrollTo/scrollBy even when scroll position does not change

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Matt Brubeck (:mbrubeck)]

Attachment: test case: onload="scrollTo(0,-1)"

When content calls window.scrollTo/scrollBy in Fennec, we would like to hide the Fennec chrome.  This is useful for pages like Google Maps which use this to hide the titlebar and sidebars on page load (see bug 653956).

In the default Android and iOS browsers, calling scrollTo causes the browser chrome to scroll out of view even if it does not change the content scroll position.  For example, the attached test case calls scrollTo(0,-1) on load, and it hides the browser chrome in the Android Browser but not in Fennec.

nsGfxScrollFrameInner::ScrollToImpl suppresses all events and notifications if the new location is the same as the old one.  We would need some sort of notification that still fires in this case.  I hope we can do this without a noticeable performance impact from too many messages:
http://mxr.mozilla.org/mozilla-central/source/layout/generic/nsGfxScrollFrame.cpp#1828

Comment 1

[Matt Brubeck (:mbrubeck)]

Nominating for tracking-fennec because it would improve usability on popular pages like Google Maps.

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Still tracking, but it won't make fx7, let alone fx6, unless some work starts soon.

Comment 3

[Aaron Train [:aaronmt]]

Is this still an issue on Fennec Native? Does it still block 616348?

Comment 4

[Lawrence Mandel [:lmandel] (use needinfo)]

(In reply to Aaron Train [:aaronmt] from comment #3)
> Is this still an issue on Fennec Native? Does it still block 616348?

Mark/Matt - Can one of you please comment to answer Aaron's question.

Comment 5

[Kevin Brosnan [Ex-Mozilla]]

Mat we believe that this was fixed by the native rewrite. Reopen if this is still an issue.