1131345 - GetVariable method no longer works for windowless 0-size SWFs

Status: RESOLVED INCOMPLETE

(Core Graveyard :: Plug-ins, defect)

Description

[T. Brains]

An SWF which is windowless (wmode either "opaque" or "transparent") and has 0px width or height, always returns "undefined" to any GetVariable call.

This is a regression in Firefox 36 beta, and works fine in Firefox 35 and previous versions, as well as in all other browsers.

Comment 1

[Matthias Versen [:Matti]]

Can you provide a testcase either attached or as URL ?

Comment 2

[T. Brains]

As you can see here: http://brainy.dagon.org/b1131345.html

All of these should return "bar", but on Firefox 36, the 0x0 transparent/opaque object/embed return null instead.

Comment 3

[Matthias Versen [:Matti]]

12:53.38 LOG: MainThread Bisector INFO Last good revision: 52156df5e298
12:53.38 LOG: MainThread Bisector INFO First bad revision: 0a0e57c4e420
12:53.38 LOG: MainThread Bisector INFO Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=52156df5e298&tochange=0a0e57c4e420

Makoto Kato — Bug 1120747 - dom.ipc.plugins.flash.disable-protected-mode might not work when launching Flash is immediately. r=aklotz

Seems to be a cause of the protected mode....

Comment 4

[Lawrence Mandel [:lmandel] (use needinfo)]

Tracking 36+ as that is when the regression was introduced.

Makoto - Given that bug 1120747 was called out as the cause in comment 3, can you take this bug?

Comment 5

[Benjamin Smedberg]

This appears to be caused by disabling protected mode (bug 1119941), which is reverted in beta9 (see bug 1120993).

This is unexpected and needs diagnosis. I'll take this for now.

Comment 6

[Benjamin Smedberg]

T. Brains, could I see the source code of that .swf?

Comment 7

[Benjamin Smedberg]

Is .getVariable("foo") supposed return "bar" in every condition?

Comment 8

[T. Brains]

Yeah, the source code of the SWF is simply setting a variable named "foo" with a value "bar" as soon as it initializes. As long as the SWF has initialized, the GetVariable("foo") call should always return bar.

I can't tell if the problem is that the SWF doesn't initialize at all, or if the problem is with the GetVariable call itself.

Comment 9

[T. Brains]

Do you require any more information from me on this? Is there any chance this will be fixed before 36 becomes production? We need to evaluate the situation here because it breaks our production code.

Comment 10

[Benjamin Smedberg]

Protected mode was only a test on beta, it is not going to release by default.

I tried to reproduce this using my own SWF (built in Flash builder) and was unable to reproduce. Could you send me the exact source code of the SWF testcase?

Comment 11

[T. Brains]

This is an AS2 SWF built using FlashDevelop and MTASC. The source code is as following:

class Main
{
	static function main(timeline:MovieClip) {
		timeline.foo = "bar";
	}
}

Comment 12

[T. Brains]

Attachment: FlashDevelop project as a testcase

I've attached a zip of a FlashDevelop project you can use to build this SWF.

Comment 13

[Lawrence Mandel [:lmandel] (use needinfo)]

As we're not going to disable protected mode automatically, can resolve this bug?

Comment 14

[Benjamin Smedberg]

Yeah, we're not going to do this automatically and I don't think we'd spend additional diagnostic time on it because of that.