646270 - MozScrolledAreaChanged changed behavior and broke some browser-chrome tests

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

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

Attachment: patch

"MozScrolledAreaChanged" is now always fired before "pageshow" for initial page sizing. That breaks some test code that used the message as a post-pageshow helper. I ditched that code and reverted back to simpler code to get tests green on linux desktop.

I also turned off a scrollbar tests. It seems we will grow the height of a document based on the width to ensure zooming out to the width will always have document available height-wise. This breaks a scrollbar test that assumes the vertical scrollbar will not be visible.

Linux desktop tests are green with patch.

We could change our behavior to not over extend the height unless we actually need to, dynamically based on current zoom. Filing a bug for that.

Comment 1

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

pushed:
http://hg.mozilla.org/mobile-browser/rev/3f9373ef450e

Comment 2

[Dão Gottwald [:dao]]

bugspam