1283274 - Avoid the wmode windowed -> opaque Flash workaround when we detect Flash async drawing support

Status: RESOLVED FIXED

(Core Graveyard :: Plug-ins, defect, P1)

Description

[Jim Mathies [:jimm]]

In newer versions of Flash that have support, Flash will convert windowed mode plugins to a new windowless async rendering model. This switch will be opaque to Firefox.

Firefox currently forces windowless in a few cases to avoid windowed plugin issues which this new mode addresses, so we can stop doing this for this new version of Flash.

Here's the workaround code in Firefox - 

http://searchfox.org/mozilla-central/rev/970569ad57ac4436ff31aa2ac63f38ed3ee2932d/dom/plugins/ipc/PluginModuleParent.cpp#2680-2700

To detect the new Flash mode, we'll need to search the dll's version info table for a specific string resource - 

\StringFileInfo\040904E4\AsyncDrawingSupport

value of “1” indicates support, no entry or a value of "0" indicates no support.

Comment 1

[Brad Lassey [:blassey] (use needinfo?)]

[Tracking Requested - why for this release]: We should get this into 48, let's track for that

Comment 2

[Liz Henry (:lizzard) (relman/hg->git project)]

OK, tracking as we discussed in irc, since 48 and onwards may be affected.

Comment 3

[Jim Mathies [:jimm]]

Attachment: patch

Comment 4

[Jim Mathies [:jimm]]

Attachment: patch

Comment 5

[Jim Mathies [:jimm]]

Attachment: patch

We need to check flash dlls for a string that may or may not be present regardless of version. We have access to the dll when we refresh pluginreg.dat and when we load the library to instantiate an instance. I'm purposely avoiding attempting to store this information in pluginreg.dat or propagate it around with the plugin tag info to keep the patch simple for uplift.

Comment 6

[Brad Lassey [:blassey] (use needinfo?)]

Comment on attachment 8773296 [details] [diff] [review]
patch

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

::: dom/plugins/base/nsPluginsDir.h
@@ +33,5 @@
>  	char** fExtensionArray;
>  	char* fFileName;
>  	char* fFullPath;
>  	char* fVersion;
> +  bool fSupportsAsyncRender;

ws

Comment 7

[Jim Mathies [:jimm]]

try push:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=f214a061f83f

Comment 8

[Masatoshi Kimura [:emk]]

Will IME work if Flash supports async rendering?

Comment 9

[Jim Mathies [:jimm]]

Attachment: patch

minor touch up on some ifdef'ing here which broke on linux.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=3f7be7fd493560cea62192acabbc2bcb766b3378

Comment 10

[Jim Mathies [:jimm]]

(In reply to Masatoshi Kimura [:emk] from comment #8)
> Will IME work if Flash supports async rendering?

I assume so. I have the beta installed and I can add an IME, can you give me some guidance on testing?

Comment 11

[Masatoshi Kimura [:emk]]

(In reply to Jim Mathies [:jimm] from comment #10)
> I assume so. I have the beta installed and I can add an IME, can you give me
> some guidance on testing?

1. Install Japanese language pack to add an IME.
2. Open <https://emk.name/test/swftxt.html>.
3. Press Alt+[`] (if US keyboard) on the Flash text box.
4. Type [a] and [Enter]. "あ" will be displayed if the IME works.

Comment 12

[Masatoshi Kimura [:emk]]

1.5. Switch the keyboard layout to Japanese.

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0474eda33487
Disable windowless workaround if we detect a flash library that support async rendering. r=aklotz

Comment 14

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/0474eda33487

Comment 15

[Jim Mathies [:jimm]]

Comment on attachment 8774533 [details] [diff] [review]
patch

Approval Request Comment
[Feature/regressing bug #]:
Enable new async drawing feature available in flash rev 23.
[User impact if declined]:
windowed plugins that could be removed from our content view remain. (which sucks!)
[Describe test coverage new/current, TreeHerder]:
Currently on nightly, tested by me, I've also asked Adobe to take nightly for a spin to check for issues.
[Risks and why]:
Enables a new drawing mode so there is some risk of fallout. The new Flash feature has been in development for around seven months.
[String/UUID change made/needed]:
none.

Comment 16

[Masatoshi Kimura [:emk]]

Neither wmode=window nor wmode=opaque worked for me with Nightly+Flash Player 23.0.0.111 beta. Even ASCII input did not work. Only wmode=transparent worked. Is this a beta issue?

Comment 17

[Masatoshi Kimura [:emk]]

(In reply to Masatoshi Kimura [:emk] from comment #16)
> Neither wmode=window nor wmode=opaque worked for me with Nightly+Flash
> Player 23.0.0.111 beta. Even ASCII input did not work. Only
> wmode=transparent worked. Is this a beta issue?

It was bug 1289859.

Comment 18

[Jim Mathies [:jimm]]

(In reply to Masatoshi Kimura [:emk] from comment #17)
> (In reply to Masatoshi Kimura [:emk] from comment #16)
> > Neither wmode=window nor wmode=opaque worked for me with Nightly+Flash
> > Player 23.0.0.111 beta. Even ASCII input did not work. Only
> > wmode=transparent worked. Is this a beta issue?
> 
> It was bug 1289859.

Thanks, I've requesting tracking on that for 49 so it gets uplifted.

Comment 19

[Gerry Chang [:gchang]]

Comment on attachment 8774533 [details] [diff] [review]
patch

This patch fixes async drawing feature flash player issue. Take it in aurora.

Comment 20

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/698bdd627b48

Comment 21

[Jukka Jylänki]

It looks as if this fix slipped from getting included to Firefox 50.0 64-bit release. Marked that down as https://bugzilla.mozilla.org/show_bug.cgi?id=1317995.

Comment 22

[Milan Sreckovic [:milan] (needinfo for best results)]

(In reply to Jukka Jylänki from comment #21)
> It looks as if this fix slipped from getting included to Firefox 50.0 64-bit
> release. Marked that down as
> https://bugzilla.mozilla.org/show_bug.cgi?id=1317995.

Or the 64-bit specific code in this patch is the difference.

Comment 23

[Jim Mathies [:jimm]]

(In reply to Jukka Jylänki from comment #21)
> It looks as if this fix slipped from getting included to Firefox 50.0 64-bit
> release. Marked that down as
> https://bugzilla.mozilla.org/show_bug.cgi?id=1317995.

Here's the code - 

https://dxr.mozilla.org/mozilla-release/rev/e5e59c346966c94f21ff5bf9033b1f08b30ed077/dom/plugins/ipc/PluginModuleParent.cpp#2717

On 64-bit firefox checks the async pref (which is false) then checks for a wmode value that isn't "transparent" and convert it if needed.