Closed Bug 524695 Opened 10 years ago Closed 10 years ago

Unmatched beginBatchOperation: closing pages that are still loaded

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
x86
macOS
Type:
defect
Priority:
Not set

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
fennec1.0b5

People

(Reporter: stechz, Assigned: stechz)

Details

Attachments

(1 file, 1 obsolete file)

Remove if from beginLoading as well
1.68 KB, patch
mfinkle
: review+
Details | Diff | Splinter Review 
The timer will go on forever because tab._loading is never false.

STR:
1) Go to cnn.com and click an ad.
2) Close the newly opened tab before it is finished loading
3) Pan around on cnn.com and find a gray square.  The gray boxes will only be painted every couple of seconds.
Attached patch Finish up in _destroyBrowser (obsolete) — — Splinter Review
Assignee: nobody → webapps
Status: NEW → ASSIGNED
Attachment #408619 - Flags: review?(mark.finkle)
Fixes an unmatched commitBatchOps as well.
Attachment #408619 - Attachment is obsolete: true
Attachment #408623 - Flags: review?(mark.finkle)
Attachment #408619 - Flags: review?(mark.finkle)
Attachment #408623 - Flags: review?(mark.finkle) → review+
pushed:
https://hg.mozilla.org/mobile-browser/rev/f9b06a49cc7e
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → B5
verified FIXED on builds:

Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2b2pre) Gecko/20091029 Namoroka/3.6b2pre Fennec/1.0b5pre

and

Mozilla/5.0 (X11; U; Linux armv6l; Nokia N8xx; en-US; rv:1.9.3a1pre) Gecko/20091029 Namoroka/3.7a1pre Fennec/1.0b5pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.