'Tap here to activate plugin' UI is displayed although the plugin option is set to "Enabled"

VERIFIED FIXED in Firefox 24

Status

()

VERIFIED FIXED
5 years ago
2 years ago

People

(Reporter: TeoVermesan, Assigned: benjamin)

Tracking

({regression, reproducible})

24 Branch
Firefox 24
ARM
Android
regression, reproducible
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox22 unaffected, firefox23 unaffected, firefox24+ verified)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Tested with:
Build: Firefox for Android 24.0a1 (2013-06-18)
Device: LG Nexus 4
OS: Android 4.2.2

Set plugins to 'Enabled'
Steps to reproduce:
Go to http://www.intel.com/museumofme/

Expected results:
- You should see flash plugins without having to tap

Actual results:
- The message: 'Tap here to activate plugin' appears as if plugins were set on 'Tap to Play'
(Reporter)

Updated

5 years ago
status-firefox22: --- → unaffected
status-firefox23: --- → unaffected
status-firefox24: --- → affected
Please find a regression range.
Flags: needinfo?(teodora.vermesan)
Flags: needinfo?
Keywords: regression, regressionwindow-wanted

Updated

5 years ago
tracking-fennec: --- → ?
Flags: needinfo?
Keywords: reproducible
(Reporter)

Comment 2

5 years ago
regressionwindow-wanted:
1.mozilla central:
good build: 11-06-2013
bad build:  12-06-2013

pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=86413e921d5d&tochange=0414d6d0f60d

2.inbound:
good build: 1370961674
bad build:  1370962937

pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=8ac5d1f98b21&tochange=5ef49706a56a
Flags: needinfo?(teodora.vermesan)

Updated

5 years ago
Blocks: 875454
Flags: needinfo?(benjamin)

Updated

5 years ago
Keywords: regressionwindow-wanted
(Assignee)

Comment 3

5 years ago
Is there a site-specific permission for that site? Previously we only checked site-specific permissions if a plugin was set to "click-to-enable". Now we honor site-specific permissions for enabled plugins as well. So if you had a site-specific click-to-activate permission for intel.com, that will override the normal setting.

Does fennec have an about:permissions or something like that? Does this happen on other sites?
Flags: needinfo?(benjamin)
It doesn't look like any site permissions are set for that URL. 

I also see this on other URLs such as: http://www.kace.com/demo/flashdemo and http://www.sonycreativesoftware.com/products/sdml/sdml.asp

They are all prompting for tap to activate despite our preference set as enabled.
(Assignee)

Comment 5

5 years ago
Created attachment 764820 [details] [diff] [review]
Fix the android pref settings, rev. 1

This was written blind, but it's pretty straightforward. The plugins.click_to_play preference no longer does anything, and so you should be modifying the plugin.default.state preference instead.

This will mean that users who currently have "always activate" set will need to make that change again, or we'll need to migrate the preference on upgrade. I'm not sure whether that's important or the best place to accomplish that.
Attachment #764820 - Flags: review?(margaret.leibovic)
(Assignee)

Updated

5 years ago
Assignee: nobody → benjamin
tracking-firefox24: --- → +

Comment 6

5 years ago
Comment on attachment 764820 [details] [diff] [review]
Fix the android pref settings, rev. 1

This looks good, and I confirmed it works as expected.
Attachment #764820 - Flags: review?(margaret.leibovic) → review+

Comment 7

5 years ago
(In reply to Benjamin Smedberg  [:bsmedberg] from comment #5)

> This will mean that users who currently have "always activate" set will need
> to make that change again, or we'll need to migrate the preference on
> upgrade. I'm not sure whether that's important or the best place to
> accomplish that.

I'm also not sure what we should do here. Off the top of my head, I don't know if we have a place where we do gecko pref migrations. If it's not too difficult to implement, I think it would be worthwhile to migrate the pref.

Comment 8

5 years ago
https://hg.mozilla.org/mozilla-central/rev/aa4eb6a7a300
Status: NEW → RESOLVED
Last Resolved: 5 years ago
status-firefox24: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 24

Updated

5 years ago
Depends on: 885357
(Reporter)

Comment 9

5 years ago
Verified fixed on:
Build: Firefox for Android 24.0a1(2013-06-21) 
Device: Samsung Galaxy Nexus
OS: Android 4.1.1
status-firefox24: fixed → verified
(Reporter)

Updated

5 years ago
Status: RESOLVED → VERIFIED
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.