Last Comment Bug 752746 - can't make liveconnect calls from javascript to applet's method
: can't make liveconnect calls from javascript to applet's method
Status: RESOLVED FIXED
: regression, testcase
Product: Core
Classification: Components
Component: Plug-ins (show other bugs)
: unspecified
: All All
: -- normal (vote)
: ---
Assigned To: John Schoenick [:johns]
:
: Benjamin Smedberg [:bsmedberg]
Mentors:
: 753280 (view as bug list)
Depends on: 745030
Blocks: CVE-2013-1717
  Show dependency treegraph
 
Reported: 2012-05-07 16:53 PDT by Calvin Cheung
Modified: 2012-07-19 09:36 PDT (History)
12 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---
+
unaffected
+
unaffected
unaffected


Attachments
liveconnect (javascript calling applet's method) testcase 1 (10.00 KB, application/octet-stream)
2012-05-07 16:53 PDT, Calvin Cheung
no flags Details
liveconnect (javascript calling applet's method) test case 2 (10.00 KB, application/octet-stream)
2012-05-07 16:54 PDT, Calvin Cheung
no flags Details

Description Calvin Cheung 2012-05-07 16:53:57 PDT
Created attachment 621789 [details]
liveconnect (javascript calling applet's method) testcase 1

User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.1; Trident/4.0; .NET CLR 1.1.4322; .NET4.0C; .NET4.0E; BRI/2; .NET CLR 2.0.50727)

Steps to reproduce:

Using the nightly 15.0a1 (2012-05-07) from http://nightly.mozilla.org/.
Tested on windows XP with JRE 7u4 (downloaded from java.com).

Ran some liveconnect (javascript calling into applet's method) testcases.

I'm attaching 2 testcases.
Both are liveconnect calls from javascript into applet's method.
The class files were compiled with jdk7.

StringTest - calls into applet's method upon "OnLoad"
JSJavaTest1 - calls into applet's method upon clicking on the javascript
button.

My observation is that the java plugin module (npjp2.dll) wasn't loaded when
running the above tests.




Actual results:

Nothing happened.


Expected results:

Liveconnect calls should be successful.
Comment 1 Calvin Cheung 2012-05-07 16:54:47 PDT
Created attachment 621790 [details]
liveconnect (javascript calling applet's method) test case 2
Comment 2 Boris Zbarsky [:bz] (still a bit busy) 2012-05-07 22:19:38 PDT
Requesting tracking for the regression.
Comment 3 Olli Pettay [:smaug] 2012-05-08 02:37:53 PDT
Assigning back to Josh
Comment 4 Josh Aas 2012-05-08 07:53:34 PDT
My debugging indicates that this is failing because we fail to create a Java plugin instance because we don't have an object frame. This makes me think this is a regression from bug 90268 or its follow-up fixes, not from the DOM object removal. The fact that this bug exists on Aurora, where we didn't remove the DOM object support, confirms my theory.
Comment 5 Josh Aas 2012-05-08 08:03:22 PDT
This works in the current Firefox 13 Beta, but not the current Firefox 14 Aurora or Firefox 15 Nightly.

This means bug 748343 isn't the regressing bug, removing it from the deps.
Comment 6 Georg Fritzsche [:gfritzsche] 2012-05-09 05:32:58 PDT
*** Bug 753280 has been marked as a duplicate of this bug. ***
Comment 7 Lukas Blakk [:lsblakk] use ?needinfo 2012-05-09 12:50:27 PDT
Tracking for 14, can we get a regression range here?
Comment 8 Josh Aas 2012-05-10 16:59:30 PDT
From hg bisect:

The first bad revision is:
changeset:   87974:1e4675fb8fc9
user:        John Schoenick <jschoenick@mozilla.com>
date:        Mon Feb 27 13:58:00 2012 -0800
summary:     Bug 406541 - Ensure we agree with java on applet codebase, and run security checks for file: codebase URIs. r=jst
Comment 9 John Schoenick [:johns] 2012-05-10 19:44:11 PDT
406541 exposed several issues in the objectloadingcontent code, 745030 should fix this
Comment 10 John Schoenick [:johns] 2012-05-10 19:45:32 PDT
Note that 406541 is going to be backed out from 14 due to all the issues depending on 745030, which will be too big a change for backporting
Comment 11 Scoobidiver (away) 2012-05-10 22:40:25 PDT
This bug blocks bug 406541 and doesn't depend on it. Can someone change the dependency?
Comment 12 Alex Keybl [:akeybl] 2012-06-14 08:50:31 PDT
(In reply to John Schoenick [:johns] from comment #10)
> Note that 406541 is going to be backed out from 14 due to all the issues
> depending on 745030, which will be too big a change for backporting

John - are you planning to fix this for FF14 (shipping in 4 weeks), or would you like to make the case for untracking? We haven't seen a status update here in a while.
Comment 13 John Schoenick [:johns] 2012-06-14 12:10:17 PDT
This is a similar regression to bug 736965, so my comments there apply - bug 745030 is the proper fix for this, but 406541 will be backed out of the affected versions until 745030 is ready (a rather large patch). 

This could probably be duped against bug 736965, but I'd like to keep it open to make sure we've covered all known regression issues. It is probably redundant to track it, however
Comment 14 John Schoenick [:johns] 2012-07-19 09:36:11 PDT
Bug 406541 has been backed out of all branches, so this regression should be no more

Note You need to log in before you can comment on or make changes to this bug.