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

RESOLVED FIXED

Status

()

defect
P1
normal
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: pcwalton, Assigned: dougt)

Tracking

unspecified
x86
macOS
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

()

Attachments

(1 attachment, 1 obsolete attachment)

Reporter

Description

8 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...)
Assignee

Comment 1

8 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

8 years ago
Posted patch patch v.1 (obsolete) — Splinter Review
Attachment #577481 - Flags: review?(mark.finkle)
Assignee

Comment 3

8 years ago
Posted patch patch v.1Splinter Review
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

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