Bug 8636

Summary: REGRESSION: JavaScript access to Java applet causes hang (_webViewURL not implemented)
Product: WebKit Reporter: Alex Taylor <darwin>
Component: JavaAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: alice.barraclough, ap, darin, ggaren, mjs, timo
Priority: P1 Keywords: InRadar, Regression
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://duckware.com/label/index.html
Attachments:
Description Flags
ZIP Archive containing HTML and Java applet
none
Console output
none
Java log file
none
Crash Report
none
Patch ggaren: review+

Alex Taylor
Reported 2006-04-27 16:10:57 PDT
In a page where an applet's method is envoked from Javascript, the method fails to execute and causes a subsequent page unload (navigating away from the page, closing the window) to crash WebKit. Reduced test case below including the applet in question, URL for reference only. Works fine in 417.9.2, haven't looked for the build where the regression occured.
Attachments
ZIP Archive containing HTML and Java applet (75.47 KB, application/zip)
2006-04-27 16:13 PDT, Alex Taylor 		no flags
Details
Console output (1.99 KB, text/plain)
2006-04-27 16:28 PDT, Alex Taylor 		no flags
Details
Java log file (17.39 KB, text/plain)
2006-04-29 13:48 PDT, Alex Taylor 		no flags
Details
Crash Report (37.06 KB, text/plain)
2006-04-29 13:51 PDT, Alex Taylor 		no flags
Details
Patch (2.62 KB, patch)
2006-06-27 16:29 PDT, Anders Carlsson 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alex Taylor
Comment 1 2006-04-27 16:13:50 PDT
Created attachment 8009 [details] ZIP Archive containing HTML and Java applet Archive contains the applet (in case it is important) and an HTML page to repro the bug.
Alex Taylor
Comment 2 2006-04-27 16:15:29 PDT
Steps to reproduce: (After unarchiving reduced testcase) 1) Load page 2) Click hyperlink to execute javscript method on Applet 3) Navigate away from page or close window/tab Actual Result: Crash Expected Result: Java method envoked
Alex Taylor
Comment 3 2006-04-27 16:27:37 PDT
*** Bug 8635 has been marked as a duplicate of this bug. ***
Alex Taylor
Comment 4 2006-04-27 16:28:49 PDT
Created attachment 8010 [details] Console output Console output from page load until after crash.
Alex Taylor
Comment 5 2006-04-27 16:36:17 PDT
Setting P1 for reproducable Crash/Hang. Priority=norm for later triage. Added Keyword 'Regression'.
Geoffrey Garen
Comment 6 2006-04-27 16:43:23 PDT
-[JavaWebKitView _webViewURL]: selector not recognized Looks like we're calling a method that's not implemented by the Java Plug-In. Might be an SPI weening global replace gone awry. Tim?
Alex Taylor
Comment 7 2006-04-29 13:48:05 PDT
Created attachment 8039 [details] Java log file Java log file for crash
Alex Taylor
Comment 8 2006-04-29 13:51:08 PDT
Created attachment 8040 [details] Crash Report Safari/Webkit crash report
Alexey Proskuryakov
Comment 9 2006-05-03 05:02:24 PDT
Cannot confirm yet, since the test case just crashes (in freeWidthMap(), this is a known issue).
Alexey Proskuryakov
Comment 10 2006-05-06 00:23:05 PDT
With r14184, I'm getting the same stack trace as in bug 8717 (ATSUConvertCharToGlyphs at the top).
Darin Adler
Comment 11 2006-06-04 16:07:13 PDT
JavaScriptCore calls the method _webViewURL. This method was implemented in WebKit until revision 14302, on 2006-04-24, for bug 8562. The call to the method is still present in JavaScriptCore but we have removed the implementation of it. I believe that may be the source of this bug and of bug 8717 as well.
Alice Liu
Comment 12 2006-06-06 06:59:56 PDT
<rdar://problem/4575074>
Anders Carlsson
Comment 13 2006-06-27 16:29:05 PDT
Created attachment 9070 [details] Patch
Geoffrey Garen
Comment 14 2006-06-27 16:32:02 PDT
Comment on attachment 9070 [details] Patch r=me, assuming this passes the tests in <rdar://3863248>
Anders Carlsson
Comment 15 2006-06-27 16:54:33 PDT
Fixed in r15070
Note You need to log in before you can comment on or make changes to this bug.