Closed Bug 920947 Opened 11 years ago Closed 10 years ago

Need to remove plugin doorhanger when last CTP plugin is removed from document

Categories

(SeaMonkey :: General, defect)

defect
Not set
normal

Tracking

(seamonkey2.22 affected, seamonkey2.23 affected, seamonkey2.24 affected)

RESOLVED WONTFIX
Tracking Status
seamonkey2.22 --- affected
seamonkey2.23 --- affected
seamonkey2.24 --- affected

People

(Reporter: neil, Assigned: neil)

Details

Attachments

(1 file)

Steps to reproduce problem: 1. Visit page with CTP plugin 2. Page removes plugin element 3. Click doorhanger Attachment 770720 [details] works if you have Java installed, but you can easily tweak it to use a different plugin.
Attached patch Proposed patchSplinter Review
PluginRemoved was added by bug 889788. This patch avoids the bug 916542 issue by explicitly looking for a click to play plugin, which means that we also handle the edge case of removing the last such while playing plugins are still present.
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #810466 - Flags: review?(philip.chee)
Comment on attachment 810466 [details] [diff] [review] Proposed patch r=me
Attachment #810466 - Flags: review?(philip.chee) → review+
Firefox Bug 889788 landed on mozilla25. We need to land this patch on aurora and beta.
Given bug 937616, this is now WONTFIX, I think?
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: