Closed Bug 789620 Opened 12 years ago Closed 12 years ago

click-to-play: a plugin made click-to-play by the blocklist won't go back to normal if unblocked

Categories

(Core Graveyard :: Plug-ins, defect)

defect
Not set
normal

Tracking

(firefox17 fixed, firefox18 fixed)

RESOLVED FIXED
mozilla18
Tracking Status
firefox17 --- fixed
firefox18 --- fixed

People

(Reporter: keeler, Assigned: keeler)

References

Details

Attachments

(1 file)

So, it turns out if we were to blocklist a plugin but later undo that block, the plugin would stay click-to-play.
Attached patch patchSplinter Review
Attachment #659409 - Flags: review?(bmcbride)
Comment on attachment 659409 [details] [diff] [review]
patch

Review of attachment 659409 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/mozapps/extensions/nsBlocklistService.js
@@ +970,5 @@
>          plugin.blocklisted = state == Ci.nsIBlocklistService.STATE_BLOCKED;
>          if (state == Ci.nsIBlocklistService.STATE_VULNERABLE_UPDATE_AVAILABLE ||
>              state == Ci.nsIBlocklistService.STATE_VULNERABLE_NO_UPDATE)
>            plugin.clicktoplay = true;
> +        // turn off clicktoplay if it was previously set by the blocklist

Hmm, just realized this is going to be a pain when we expose UI for click-to-play for individual plugins (bug 549697), since this will undo the user's choice. Before we get that UI, I think we'll have to separate out blocklist-setting and user-setting for clicktoplay. Shouldn't be too painful - change usage here to .blocklistClicktoplay, add a .userClicktoplay, and make .clicktoplay only an accessor property whose value is based on those two.
Attachment #659409 - Flags: review?(bmcbride) → review+
Status: NEW → ASSIGNED
Keywords: checkin-needed
Flags: in-testsuite+
https://hg.mozilla.org/mozilla-central/rev/74d24d902ad6
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
Comment on attachment 659409 [details] [diff] [review]
patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): blocklist/click-to-play plugins (bug 760625)
User impact if declined: if a plugin is click-to-play-blocklisted and then removed from the blocklist, it will continue to be a click-to-play plugin (with no UI to make it go back to normal)
Testing completed (on m-c, etc.): tested on m-c, been on nightly for a few weeks
Risk to taking this patch (and alternatives if risky): no particular risk
String or UUID changes made by this patch: none
Attachment #659409 - Flags: approval-mozilla-aurora?
Comment on attachment 659409 [details] [diff] [review]
patch

[Triage Comment]
Good fix for a newly uplifted feature.
Attachment #659409 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Keywords: checkin-needed
Whiteboard: [patch needs uplift to Aurora]
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: