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)

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

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.
Status: NEW → ASSIGNED
Setting all current Open/Normal to M4.
per leger, assigning QA contacts to all open bugs without QA contacts according to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
QA Contact: 3849 → 4082
reassigning Greg Lynn as QA contact
QA Contact: 4082 → 4015
Alex any update on this bug? QA assigned to whitebox, gerardok
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.
Rick - who owns the webshell code?
Target Milestone: M4 → M5
Priority: P2 → P1
Nisheeth - can you take a look at this?
Target Milestone: M5 → M6
Whiteboard: need help from nisheeth?
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
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!
Seems to work for me now.
Status: RESOLVED → VERIFIED
Whiteboard: need help from nisheeth?
verified by reporter
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.