Page only paints once during load

VERIFIED FIXED

Status

Fennec Graveyard
General
VERIFIED FIXED
7 years ago
4 years ago

People

(Reporter: stechz, Assigned: stechz)

Tracking

Bug Flags:
in-testsuite ?

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

7 years ago
Due to a bug in Util.Timeout (see https://bugzilla.mozilla.org/show_bug.cgi?id=574117#c4).  Patch coming soon.

Comment 1

7 years ago
Flagging for 2.0 triage
tracking-fennec: --- → ?
(Assignee)

Comment 2

7 years ago
Created attachment 466339 [details] [diff] [review]
Patch
Assignee: nobody → webapps
Attachment #466339 - Flags: review?(mark.finkle)
Comment on attachment 466339 [details] [diff] [review]
Patch

Wrong patch?
(Assignee)

Comment 4

7 years ago
Created attachment 466541 [details] [diff] [review]
The real patch
Attachment #466541 - Flags: review?(mark.finkle)
(Assignee)

Updated

7 years ago
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+
Created attachment 466682 [details] [diff] [review]
bustage-fix

This patch fix the context menu bustage observe since this patch because TYPE_ONE_SHOT is equal 0
Attachment #466682 - Flags: review?
Attachment #466682 - Flags: review? → review?(webapps)
(Assignee)

Comment 7

7 years ago
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's push:
http://hg.mozilla.org/mobile-browser/rev/b769b2f7fd9e

little fix:
http://hg.mozilla.org/mobile-browser/rev/47e02fd5f272
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
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.