Nothing happens when opening a .swf flash file directly

RESOLVED FIXED in Firefox 11

Status

()

Core
Plug-ins
P2
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Martijn Wargers (dead), Assigned: Margaret)

Tracking

({testcase})

Trunk
mozilla12
ARM
Android
testcase
Points:
---

Firefox Tracking Flags

(firefox11 fixed, firefox12 fixed, fennec11+)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
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.
(Reporter)

Comment 1

5 years ago
Actually, this is probably the same issue as bug 718983.
Depends on: 718983
Assignee: nobody → snorp
tracking-fennec: --- → 11+
Priority: -- → P2
(Assignee)

Comment 2

5 years ago
(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!
(Assignee)

Updated

5 years ago
Duplicate of this bug: 718983
(Assignee)

Updated

5 years ago
No longer depends on: 718983
(Assignee)

Comment 5

5 years ago
Created attachment 591336 [details] [diff] [review]
patch

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+
(Assignee)

Comment 7

5 years ago
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

Comment 8

5 years ago
https://hg.mozilla.org/mozilla-central/rev/d543ed92dcbd
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
(Assignee)

Comment 9

5 years ago
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+
(Assignee)

Comment 11

5 years ago
https://hg.mozilla.org/releases/mozilla-aurora/rev/e71036b16553
status-firefox11: --- → fixed
status-firefox12: --- → fixed
You need to log in before you can comment on or make changes to this bug.