Closed Bug 712414 Opened 10 years ago Closed 9 years ago
Throbber does not stop after running on pages that use document
.write without document .close
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.
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+
9 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
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
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?
Comment on attachment 583508 [details] [diff] [review] patch [Triage Comment] Mobile only - approved for Aurora.
Attachment #583508 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.