Closed
Bug 11953
Opened 26 years ago
Closed 26 years ago
[WebShell/Necko] OnEndDocumentLoad not firing
Categories
(Core :: Layout, defect, P3)
Tracking
()
VERIFIED
FIXED
M10
People
(Reporter: morse, Assigned: rpotts)
References
Details
The OnEndDocumentLoad handler in extensions/wallet/src/nsWalletService.cpp is
never being entered. This breaks single signon's prefill which is dependent on
this handler.
Reporter | ||
Updated•26 years ago
|
Summary: OnDocumentEnd not firing → OnEndDocumentLoad not firing
Updated•26 years ago
|
Assignee: pollmann → nisheeth
Comment 1•26 years ago
|
||
I could be wrong, but this sounds like a webshell issue. Nisheeth, are you
still in charge of that area or has Scott taken it over?
Comment 2•26 years ago
|
||
Yes, Scott is the official webshell owner post-M9. Steve, I'm really busy
preparing for an XML demo on Friday and won't be able to take a look at this
bug for this week. Scott, do you have cycles that you could spend on this bug?
If so, please re-assign this to yourself. Also, ccing Rick Potts who said that
he would be working on streamlining document load notifications. Rick, this
looks like a regression due to Necko.
Updated•26 years ago
|
Status: NEW → ASSIGNED
Summary: OnEndDocumentLoad not firing → [WebShell/Necko] OnEndDocumentLoad not firing
Comment 3•26 years ago
|
||
reassigning to scc for triage, cc jevering. Should we hold M9 for this?
Assignee: nisheeth → scc
Status: ASSIGNED → NEW
Target Milestone: M9
Updated•26 years ago
|
Assignee: scc → rpotts
Comment 4•26 years ago
|
||
Warren sez rick potts is the right person to look at this.
The EndDocumentLoad notification is fired, but never makes it all the way to the topmost (service) docloader
for dissemination to its observers because some (Necko) loadgroup is still busy at the time. In fact, it never goes
unbusy. Loadgroups look to be Warren's, so I'm wondering if he wants this bug himself. Nevertheless
leaving assignee alone for now, though I'm adding him to the spamlist.
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M9 → M10
Assignee | ||
Comment 6•26 years ago
|
||
This "should" be fixed on the M9 branch... There is no test case, so I can't
verify this :-( According to the PR_LOG messages OnEndDocumentLoad is firing
all the way up to the global document loader service...
I'll keep it open until I merge these changes back onto the tip.
-- rick
Reporter | ||
Comment 7•26 years ago
|
||
Want a test case? Just set a breakpoint on the OnEndDocumentLoad handler in
extensions/wallet/src/nsWalletSerice.cpp. You should get to that breakpoint
each time you load in a page. But currently you are never getting to it.
Comment 8•26 years ago
|
||
For the record, I can confirm that this is fixed on the M9 Branch.
Assignee | ||
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•26 years ago
|
||
I've landed the M9 branch...
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Comment 10•26 years ago
|
||
I verified this on today's tip builds.
You need to log in
before you can comment on or make changes to this bug.
Description
•