519948 - showing the urlbar should not scroll to the top

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch

Steps to reproduce:
 * go to google.com
 * do any search
 * scroll to the bottom of the page
 * hit Ctrl+L
 * dismiss the urlbar

Actual result:
 * we can see the top of the web page

Expected result:
 * the part of the page we have leave before hitting Ctrl+L


I've add gFocusManager as a global var cause in a near future there will be probably more and more place where we want to use it. (e.g the scroll patch that have been back out recently, the forms helper).

This is also good for user perception when he hit Ctrl+L after panning to the bottom cause it replace :
 * scroll to the top and show a part of the urlbar
 * display the awesome list

by:
 * display the awesome list

Comment 1

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

pushed:
https://hg.mozilla.org/mobile-browser/rev/7b95ba5d1033

Comment 2

[Aakash Desai [:aakashd]]

verified FIXED On builds:

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/20091013
Fennec/1.0b5pre

and

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.3a1pre) Gecko/20091013
Fennec/1.0b5pre