Closed
Bug 2424
Opened 26 years ago
Closed 25 years ago
Full-screen plug-in: throbber never stops
Categories
(Core Graveyard :: Plug-ins, defect, P1)
Tracking
(Not tracked)
VERIFIED
FIXED
M6
People
(Reporter: cohn, Assigned: amusil)
Details
A full-screen plug-in should be an nsIDocumentViewer and support
kIDocumentViewerIID. I noticed this because the throbber never stops on a
full-screen plug-in. Stopping depends on getting an nsIDocumentViewer, which the
PluginViewerImpl is not.
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Comment 2•26 years ago
|
||
per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
Updated•26 years ago
|
QA Contact: 3849 → 4082
Comment 3•26 years ago
|
||
reassigning Greg Lynn as QA contact
Assignee | ||
Comment 5•26 years ago
|
||
It is true that the throbbing code only gets called if the viewer implements the
nsIDocumentViewer interface, but I do not think the solution is to make the
plugin viewer implement this (it is intended for a viewer that uses the layout
enginer to layout data). An nsIContentViewer derived viewer should also stop
the throbber. The relavent code is in nsWebShell::OnConnectionsComplete().
I'll figure out who to assign this to (it's whoever owns the webshell code).
There is also a crash that occurs in this same function (bug 1157) and it may be
related.
Assignee | ||
Updated•25 years ago
|
Priority: P2 → P1
Assignee | ||
Comment 7•25 years ago
|
||
Nisheeth - can you take a look at this?
Assignee | ||
Updated•25 years ago
|
Target Milestone: M5 → M6
Updated•25 years ago
|
Whiteboard: need help from nisheeth?
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 8•25 years ago
|
||
Someone re-coded the logic for the dispatch of the all connections complete
notification and this is not fixed.
If you leave bugs alone long enough, they just fix themselves... :-)
cohn, please could you verify this bug or provide a testcase to verify it.
Thanks!
Reporter | ||
Comment 10•25 years ago
|
||
Seems to work for me now.
Comment 11•25 years ago
|
||
verified by reporter
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•