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
6 years ago
6 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

6 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

6 years ago
Whiteboard: [fennec-aurora]
https://hg.mozilla.org/integration/mozilla-inbound/rev/c82245ed1e82

Comment 4

6 years ago
https://hg.mozilla.org/mozilla-central/rev/c82245ed1e82
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12

Comment 5

6 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

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

Comment 9

6 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.