plugins.click_to_play;true completely hides Java plugin

VERIFIED FIXED in mozilla17

Status

()

Core
Plug-ins
VERIFIED FIXED
5 years ago
4 years ago

People

(Reporter: Joergen Ramskov, Assigned: johns)

Tracking

14 Branch
mozilla17
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 obsolete attachment)

(Reporter)

Description

5 years ago
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:14.0) Gecko/20100101 Firefox/14.0
Build ID: 20120605113340

Steps to reproduce:

Enabled the new "plugins.click_to_play" feature. It works fine for flash on youtube and various other pages I tested, but it doesn't work for the Java applet plugin.

In Denmark, we have something called "NemID", a 2 factor login solution used by most online banks and to logon to government websites (to report taxes, etc.). It requires Java.

Example page: http://www.e-boks.dk/redirect.aspx?logon=personal




Actual results:

The Java Applet is "hidden" - there's no "click to play" option.



Expected results:

Click to play should have given me an option to start the java applet that's used for logon to many websites.
dupe of bug 747709 or bug 746888 ?
Component: Untriaged → Plug-ins
Product: Firefox → Core
QA Contact: untriaged → plugins
Created attachment 632073 [details] [diff] [review]
proposal

This is another one of those "because the plugin isn't 'enabled', nsObjectLoadingContent uses OnStartRequest instead of SyncStartPluginInstance and gets everything wrong" bugs (e.g. bug 749455, bug 743429). I think we should add a check in LoadObject to fail early if we're loading a plugin and it's click-to-play.

(For more on specifically this bug, the issue this time is that a URI that doesn't exist is passed to LoadObject. (It looks like this URI comes from the java applet's code attribute.) Without click-to-play, the plugin gets loaded and handles the URI itself. With click-to-play, OnStartRequest sees this as a failed request and cancels the load.)
Attachment #632073 - Flags: feedback?(joshmoz)
Attachment #632073 - Flags: feedback?(joshmoz)
Blocks: 738698

Comment 3

5 years ago
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:14.0) Gecko/20100101 Firefox/14.0.1
Java plugin: Java(TM) Platform SE 6 U33 6.0.330.3

Following pages are not working with Firefox 14.0.1 and plugins.click_to_play=true:
http://java.com/en/download/installed.jsp
http://www.java.com/en/download/testjava.jsp

Using respectively the button "Verify Java Version" button / the link "test the currently installed version of Java", the tests fail.

There is no UI to activate the Java plugin like here:
http://javatester.org/version.html

To note also that this last page shows another minor bug: the "click to play" message is cut.

It is possible that this has been fixed in dev releases (aurora, beta or nightly). I don't use them.
Bug 745030 fixes this. This bug should probably remain open until we're sure it won't get backed out, though.

http://www.java.com/en/download/testjava.jsp works as expected on Nightly.
http://java.com/en/download/installed.jsp doesn't seem to have a visible plugin area, but you can activate it using the icon in the url bar.
The issue with http://javatester.org/version.html is actually a "feature" - if the plugin area is too small to display the ui, we hide the overlay. This makes it look broken, though, so maybe we'll revisit that. Again, you can again use the icon in the url bar to activate the plugin.
Depends on: 745030
Attachment #632073 - Attachment is obsolete: true
(Assignee)

Updated

5 years ago
Assignee: nobody → jschoenick
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
OS: Windows XP → All
Hardware: x86 → All
(Assignee)

Comment 5

5 years ago
Bug 745030 isn't likely getting backed out, marking fixed
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
CTP overlay is displayed and Java works fine after clicking on it.
Verified fixed on Nightly 20.0a1 (2012-12-10) Win 7 x64.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.