Closed Bug 920947 Opened 6 years ago Closed 5 years ago

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

Categories

(SeaMonkey :: General, defect)

defect
Not set

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: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.