[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Gnash-commit] [patch #9279] Patch to fix wrong behavior in getURL()/Mov

From: Nutchanon Wetchasit
Subject: [Gnash-commit] [patch #9279] Patch to fix wrong behavior in getURL()/MovieClip.getURL() on `undefined` and `null` target
Date: Sun, 5 Mar 2017 06:49:45 -0500 (EST)
User-agent: Opera/9.80 (Windows NT 5.1; U; en) Presto/2.10.229 Version/11.64

Follow-up Comment #2, patch #9279 (project gnash):

Patch 3 of 6:
See the attached `0003_correct-builtin-geturl-undefined-null-target.patch`.

Current Gnash treats `undefined` and `null` target parameter in built-in
`getURL()` function as if they were empty string

Flash Player, other the other hand, uses (SWF version-specific) string
representation of the parameter
<> as target. This behavior is
the same in both normal and FSCommand usage.

This patch fixes Gnash implementation by replacing the special case treatment
with SWF version-specific string conversion.

Corresponding 4 XFAILED check in Gnash testsuite are changed to PASSED in this
patch; resulting in 16 more PASSED checks total when counted all SWF version
and test method. (8 more PASSED in automated run, 8 more PASSED in
browser-based run)

Side note (for the record):
MakeSWF processes `getURL("loaded.html")` into ActionGetURL2 instruction that
looks more or less like `getURL("loaded.html","")`. Macromedia Flash MX 2004
also processes the call into a similar statement, though it uses an old-school
ActionGetURL instead of ActionGetURL2.

Gnash: 0.8.11dev (patched against git 144e082 22-Feb-2017)
Adobe Flash Player: (NPAPI binary)
Browser: Iceweasel 10.0.12 (debian)
System: Debian GNU/Linux 7.0 Wheezy i386

(file #39891)

Additional Item Attachment:

File name: 0003_correct-builtin-geturl-undefined-null-target.patch Size:4 KB


Reply to this item at:


  Message sent via/by Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]