The default bug view has changed. See this FAQ.

Throbber does not stop after running on pages that use document.write without document.close

VERIFIED FIXED in Firefox 11

Status

()

Firefox for Android
General
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: nhirata, Assigned: mfinkle)

Tracking

unspecified
Firefox 12
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox11 fixed, firefox12 fixed)

Details

Attachments

(1 attachment)

1. download the startup5.html
2. adb push startup5.html /sdcard/download
3. am start -a android.intent.action.VIEW -n org.mozilla.fennec/.App -d file://mnt/sdcard/download/startup5.html#63728494

Expected: throbber will stop at some point in time
Actual: the throbber keeps going

Note:
20111220 build
The build is wrong : http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-central-android-r7/

However, I cannot test to see if this still occurs because of : bug 712163 is blocking all s0/s1 testing.
Created attachment 583508 [details] [diff] [review]
patch

The startup5.html file uses document.write() but never does a document.close() and I think we never get a DOCUMENT stop notificiation.

This patch switches Fennec to use NETWORK start and stop instead of DOCUMENT start and stop. Desktop Firefox also uses NETWORK:
http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.js#4547
Assignee: nobody → mark.finkle
Attachment #583508 - Flags: review?(mbrubeck)
Attachment #583508 - Flags: review?(mbrubeck) → review+
Blocks: 710297
(Assignee)

Updated

5 years ago
Summary: Throbber does not stop after running the s0/s1 (onload local/twitter local) test pages. → Throbber does not stop after running on pages that use document.write without document.close
(Assignee)

Updated

5 years ago
Whiteboard: [fennec-aurora]
https://hg.mozilla.org/integration/mozilla-inbound/rev/c82245ed1e82
https://hg.mozilla.org/mozilla-central/rev/c82245ed1e82
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12

Comment 5

5 years ago
Why has Fennec decided to differ from Firefox here?  Firefox leaves the throbber spinning; see bug 81980.  (And see bug 605937...)
(In reply to Jesse Ruderman from comment #5)
> Why has Fennec decided to differ from Firefox here?  Firefox leaves the
> throbber spinning; see bug 81980.  (And see bug 605937...)

Jesse - ironically, we wanted to match how desktop Firefox handled the page in question.
Verified fixed on Native Fennec 12.0a1 build 20120104031035.
Status: RESOLVED → VERIFIED
Comment on attachment 583508 [details] [diff] [review]
patch

[Approval Request Comment]
We want this behavior so we don't appear to be taking forever to load
Attachment #583508 - Flags: approval-mozilla-aurora?
(Assignee)

Updated

5 years ago
status-firefox11: --- → affected
status-firefox12: --- → fixed
Whiteboard: [fennec-aurora]

Comment 9

5 years ago
Comment on attachment 583508 [details] [diff] [review]
patch

[Triage Comment]
Mobile only - approved for Aurora.
Attachment #583508 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/f09ae3fa7790
status-firefox11: affected → fixed
You need to log in before you can comment on or make changes to this bug.