Closed Bug 719860 Opened 9 years ago Closed 9 years ago
Nothing happens when opening a .swf flash file directly
You need to have the preference of Plugins set to "Tap to Play" to see this bug. See url, nothing happens when loading that url. I guess that since Flash is supported, the plugin placeholder should be shown, which you can then tap to play the Flash file. Tested on the LG Optimus Black, Android 2.2.2.
Assignee: nobody → snorp
tracking-fennec: --- → 11+
Priority: -- → P2
(In reply to Martijn Wargers [:mw22] (QA - IRC nick: mw22) from comment #1) > Actually, this is probably the same issue as bug 718983. I agree. We follow a different code path for full-page plugins like this. Snorp, I can take this, since it's a click to play issue (and I'm already looking into bug 718983).
Assignee: snorp → margaret.leibovic
(In reply to Margaret Leibovic [:margaret] from comment #2) > (In reply to Martijn Wargers [:mw22] (QA - IRC nick: mw22) from comment #1) > > Actually, this is probably the same issue as bug 718983. > > I agree. We follow a different code path for full-page plugins like this. > Snorp, I can take this, since it's a click to play issue (and I'm already > looking into bug 718983). Thanks!
Disclaimer: This patch fixes the bug, but I'm not really sure why. In bug 707886, my first patch busted full page plugin tests because we weren't automatically playing those plugins, and as a fix we decided to read the plugins.click_to_play pref in nsPluginHost::IsPluginEnabledForType to decide whether or not to automatically play those plugins. However, I'm not sure this was the right way to solve the bustage, since this patch implies that this call to IsPluginEnabledForType in nsContentDLF doesn't care if a plugin is click to play or not. I think that actually must be handled later, because with this patch applied and plugins.click_to_play set to true, you get the expected "Tap to play plugin" overlay UI.
Attachment #591336 - Flags: review?(jst)
Comment on attachment 591336 [details] [diff] [review] patch Yeah, this looks right given the setup around this code.
Attachment #591336 - Flags: review?(jst) → review+
Landed on inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/d543ed92dcbd Switching components, since this is a core issue that would affect desktop as well.
Component: General → Plug-ins
Product: Fennec Native → Core
QA Contact: general → plugins
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Comment on attachment 591336 [details] [diff] [review] patch [Approval Request Comment] With click to play plugins enabled, full page plugins don't work without this patch. Although this is core code, plugins.click_to_play is disabled on desktop, so this will not have any impact on desktop Firefox.
Attachment #591336 - Flags: approval-mozilla-aurora?
Comment on attachment 591336 [details] [diff] [review] patch [Triage Comment] Near-zero risk to desktop. Approved for Aurora.
Attachment #591336 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.