Closed Bug 587663 Opened 9 years ago Closed 9 years ago

Page only paints once during load

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: stechz, Assigned: stechz)

Details

Attachments

(2 files, 1 obsolete file)

Due to a bug in Util.Timeout (see https://bugzilla.mozilla.org/show_bug.cgi?id=574117#c4).  Patch coming soon.
Flagging for 2.0 triage
tracking-fennec: --- → ?
Attached patch Patch (obsolete) — Splinter Review
Assignee: nobody → webapps
Attachment #466339 - Flags: review?(mark.finkle)
Attached patch The real patchSplinter Review
Attachment #466541 - Flags: review?(mark.finkle)
Attachment #466339 - Attachment is obsolete: true
Attachment #466339 - Flags: review?(mark.finkle)
Comment on attachment 466541 [details] [diff] [review]
The real patch

you could rename _activeType -> _type, but it's up to you.
Attachment #466541 - Flags: review?(mark.finkle) → review+
Attached patch bustage-fixSplinter Review
This patch fix the context menu bustage observe since this patch because TYPE_ONE_SHOT is equal 0
Attachment #466682 - Flags: review?
Comment on attachment 466682 [details] [diff] [review]
bustage-fix

Oops!

> +    if (this._type != null) {

That should be "!==".  And remove the dump.
Attachment #466682 - Flags: review?(webapps) → review+
Ben, what was the use case that found this bug?
Flags: in-testsuite?
verified FIXED using webhamster.com on builds:
Mozilla/5.0 (X11; U; Linux armv71; Nokia N900; en-US; rv:2.0b4pre) Gecko/20100818 Namoroka/4.0b4pre Fennec/2.0a1pre

and

Mozilla/5.0 (Android; U; Linux armv71; en-US; rv:2.0b4pre) Gecko/20100818 Namoroka/4.0b4pre Fennec/2.0a1pre
Status: RESOLVED → VERIFIED
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.