slashdot.org doesn't fire MozScrolledAreaChanged events as the page gets bigger while loading

RESOLVED FIXED

Status

()

Firefox for Android
General
P1
normal
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: pcwalton, Assigned: dougt)

Tracking

unspecified
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
Because we resize the page only upon receiving MozScrolledAreaChanged, we end up with a page much smaller than it should be. (How did XUL Fennec do this? It must have had the same issue...)
(Reporter)

Updated

6 years ago
Blocks: 695448
(Assignee)

Comment 1

6 years ago
I think we had similar problems with the xul fennec.  whenever we did complete loading, we asked "how big are you".  A workaround, yes.
Assignee: nobody → doug.turner
Priority: -- → P1
(Assignee)

Comment 2

6 years ago
Created attachment 577481 [details] [diff] [review]
patch v.1
Attachment #577481 - Flags: review?(mark.finkle)
(Assignee)

Comment 3

6 years ago
Created attachment 577483 [details] [diff] [review]
patch v.1
Attachment #577481 - Attachment is obsolete: true
Attachment #577481 - Flags: review?(mark.finkle)
Attachment #577483 - Flags: review?(mark.finkle)
Comment on attachment 577483 [details] [diff] [review]
patch v.1


>diff --git a/mobile/android/chrome/content/browser.js b/mobile/android/chrome/content/browser.js
>   get viewport() {

>-    let doc = this.browser.contentDocument.documentElement;
>+    let cdoc = this.browser.contentDocument;

Call me crazy, but I like "doc" better than "cdoc"
Attachment #577483 - Flags: review?(mark.finkle) → review+
(Assignee)

Comment 5

6 years ago
http://hg.mozilla.org/projects/birch/rev/82f3392976b4
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
You need to log in before you can comment on or make changes to this bug.