167601 - [object] Support <param name="PLUGINURL" ...>

Status: RESOLVED FIXED

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

Description

[rubydoo123]

Submitting bug to track the coding to support pluginspage in param element.

The param element coding:
<param name="pluginspage" value="path to xpi file">

The default plug-in functionality should be the same as with the embed element
and the codebase attribute hack. This is to provide a mechanism to phase out the
codebade attribute hack.

Comment 1

[rubydoo123]

Arun corrected my mistake in requesting pluginspage instead of pluginsurl. So,
the request should be to use pluginsurl. I have updated the Summary to reflect
that change.

Arun also pointed me to an historical reference of the use:
http://developer.netscape.com/docs/manuals/htmlguid/tags14.htm#1684921

Thanks Arun!

Comment 2

[Peter Lubczynski]

Attachment: patch v.1

This patch will look for PLUGINURL (singular) in the PARAM tags of OBJECT tags
instead of CODEBASE. The default plugins are already setup to look for this.

Comment 3

[Peter Lubczynski]

According to the spec here:
http://developer.netscape.com/docs/manuals/htmlguid/tags14.htm#1684921

The name value should be singluar ___PLUGINURL__

----

Anthony, can I get a review?

Comment 4

[anthonyd]

r=anthonyd, looks good.

Comment 5

[Peter K. Sheerin]

Should both PLUGINSPAGE and PLUGINURL be supported, as the reference from Arun
about Netscape 4.x documents? That would provide the advantage of allowing an
unambiguous method of specifying whether the URI points to an XPI package or a
human-readable download instruction page.

PLUGINURL could also be used by other browsers that don't support any automatic
download/installation, and by Mozilla for those cases when the XPI installation
fails (say, because the XPI doesn't contain a plugin compatible with the user's
platform).

(From http://developer.netscape.com/docs/manuals/htmlguid/tags14.htm#1684921 )
----
PLUGINSPAGE="instrURL"
specifies the URL that contains the instructions for installing the plug-in if
it is not already installed.

PLUGINURL="pluginURL"
is the URL of a Java Archive (JAR) file, which is a compressed collection of
files that can be signed.
----

Comment 6

[rubydoo123]

Peter S: see bug 169743 (just opened for pluginspage support

Comment 7

[Johnny Stenback (:jst)]

Comment on attachment 99228 [details] [diff] [review]
patch v.1

sr=jst

Comment 8

[Peter Lubczynski]

Comment on attachment 99228 [details] [diff] [review]
patch v.1

(from comments)

Comment 9

[Peter K. Sheerin]

The relevant comment in the patch still refers to pluginspage:

+    // if we didn't find a PluginsPage param on the object tag, 
+    // there's nothing more to do here
+    if(nsPluginTagType_Object == tagType && !bHasPluginURL)
       return rv;

That should probably be changed to PluginURL to avoid confusing things...

Comment 10

[Peter Lubczynski]

comment fixed and patch checked into the trunk, marking FIXED

Comment 11

[Intermittent Failures Robot]

1 failures in 3810 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• autoland: 1

Platform breakdown:

• linux1804-64-qr: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=167601&startday=2021-01-11&endday=2021-01-17&tree=all