33333 - javascript:OpenThisUrl() crashes webclient.

Status: RESOLVED INCOMPLETE

(Core Graveyard :: Java APIs to WebShell, defect, P3)

Description

[edburns]

Load the URL above in webclient, then click on the "Correct 
http://www.mozilla.org" link.

Webclient instantly crashes:

#
# HotSpot Virtual Machine Error, EXCEPTION_ACCESS_VIOLATION
#
# Error ID: 4F533F57494E13120E43505002BE
#

abnormal program termination

Comment 1

[edburns]

I filed it, I accept it.

Comment 2

[leger]

Adding crash keyword.

Comment 3

[edburns]

*** Bug 40565 has been marked as a duplicate of this bug. ***

Comment 4

[edburns]

From: Christopher Blizzard <[e-mail]>
Sender: [e-mail]
To: [e-mail]
CC: [e-mail]
Subject: Re: javascript:openThisUrl('http://www.mozilla.org'); in embedding
Date: Mon, 08 May 2000 11:46:44 -0400
Content-Type: text/plain; charset=us-ascii
MIME-Version: 1.0

[e-mail] wrote:
> 
> Hello,
> 
> How should apps embedding mozilla deal with when the user clicks on a
> 
> javascript:openThisUrl('http://www.mozilla.org');
> 
> link?
> 
> I tried to see where this happens, and got down to nsJSProtocolHandler,
> but lost track of it when it went into JavaScript.  Can someone give me
> some pointers?
> 
> Who gets asked to create a new window?

I think that javascript: urls are kinda broken in some ways.  In any
case, there is a callback that allows you to control the creation of new
windows.  Have a look at the getNewBrowser() interface in the
nsIWebBrowserChrome interface:

/*
	Tells the implementer of this interface to create a new
webBrowserChrome
	object for it.  Typically this means the implemetor will create a new 
	top level window that is represented by nsIWebBrowserChrome.  This
	most often will be called when for instance there is a need for a new
	JS window, etc.  Soon after this new object is returned, the webBrowser
	attribute will checked, if one does not exist, one will be created and
	setWebBrowser will be called with the new widget to instantiate in this 
	new window.	
*/
nsIWebBrowser getNewBrowser(in unsigned long chromeMask);

This gets turned into this call in C++:

NS_IMETHODIMP GtkMozEmbedChrome::GetNewBrowser(PRUint32 chromeMask, 
					       nsIWebBrowser **_retval)

It gives you the chance to create a new toplevel window and surround it
with your own chrome and pass it back to the layout engine.  I'll be
adding this to the gtk embedding widget which should give you some clues
how to do it.

--Chris

-- 
------------
Christopher Blizzard
http://people.redhat.com/blizzard/
He who lives by the sword, should go out and get himself a really
nice sword.
------------

Comment 5

[edburns]

Reassigning.

Comment 6

[edburns]

Depends on bug 48227

Comment 7

[Jacob Kjome]

bug 48227 is fixed.  Can this get fixed now?

Jake

Comment 8

[Jacob Kjome]

Actually, forget what I said.  This seems to be fixed in M18 2000092808

Jake

Comment 9

[John Unruh]

Worksforme with the Win98 10/3 branch build.

Comment 10

[Ashu Kulkarni]

This is a bug opened against Webclient (part of the Blackwood project), and as 
of 10/04, the fix for this bug is still not in the mozilla/java/webclient 
codebase. 

Reopening this bug.

_Ashu

Comment 11

[Frank Tang]

I cannot reproduce the crash by any of these 4 links with my local window branch
pull on 10/16 afternoon. Is this stilla crash issue? If not, please remove crash
from the keyword.

Comment 12

[Vladimir Strigun]

I reproduce it on Linux 2.2 with latest nightly mozilla & webclient.
All four links do not proceeded.

Output for first 2 links:
JavaScript error: 
 line 0: uncaught exception: [Exception... "Failure"  code: "-2147467259" nsresu
lt: "0x80004005 (NS_ERROR_FAILURE)"  location: "http://java.sun.com/people/edbur
ns/work/jsOpenThisURL.html Line: 5"]

Output for last 2 links:
# # An unexpected exception has been detected in native code outside the VM.# 
Program counter=0x499c0a77
#
# Problematic Thread: prio=1 tid=0x81ba228 nid=0xf2a runnable 
#

Comment 13

[geetha.vaidyanaathan]

Changing QA contact

Comment 14

[Ashu Kulkarni]

*** Bug 77552 has been marked as a duplicate of this bug. ***

Comment 15

[Vladimir Strigun]

On NT webclient crashed if i try to load this URL.
Step to reproduce:
1. run webclient
2. paste url to location string

Expected result : webclient load this page
Actual result: webclient crashed.
I have following output:

debug: edburns: Currently Viewing: http://java.sun.com/people/edburns/work/jsOpe
nThisURL.html
Enabling Quirk StyleSheet
debug: edburns: STATE_TRANSFERRING

Comment 16

[Oleg Khokhlov]

Current status:
Webclient loads the page and doesn't crash.
Clicking on links doesn't work but webclient doesn't crash.

WinNT 4.0 SP4 with JDK 1.4.0 beta, Netscape61_RELEASE mozilla branch + 
JAVADEV_6_1_20010831 blackwood branch;

x86 Solaris 2.8 with JDK 1.4.0-beta3-b84, MOZILLA_0_9_4_BRANCH + blackwood 
trunk;

Comment 17

[Moiz Dohadwala]

wfm build 2002053012, win2k sp2

Comment 18

[Nick]

running webclient 1.1, mozilla 0.9.2 on a Linux and javascript:document.write
("Sample message") works but javascript:window.print() does not.

Comment 19

[Juan E Erbes]

I got the crash in the java loading process with:
http://adsl.internode.on.net/speedtest/
http://www.unitedlinux.com/
Mozilla 1.5b
Mozilla/5.0 (X11; U; Linux i586; en-US; rv:1.5b) Gecko/20030908

Comment 20

[Kurt Weinschenker]

Links are dead; why not close this one, unless a test URL or testcase can be made?

Comment 21

[timeless]

closing bugs filed by active contributors is *not* recommended practice.

Comment 22

[Wayne Mery (:wsmwk)]

WUW the conflicting reports of working for some and not others?

Comment 23

[Lucas Adamski [:ladamski]]

Closing until we can find a working testcase.