Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 332917

Summary: XULRunner path may not work for SWT browser
Product: [Eclipse Project] Platform Reporter: wuzhe <wuzhe>
Component: SWTAssignee: Grant Gayed <grant_gayed>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: eclipse.felipe, liujuny
Version: 4.0   
Target Milestone: 3.7 M5   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
patch
none
applied patch none

Description wuzhe CLA 2010-12-19 21:39:26 EST
Build Identifier: 3.4.2

If add -Dorg.eclipse.swt.browser.XULRunnerPath=${rcp.home}/rcp/eclipse/plugins/xulrunner in rcpinstall.properties. Because of the mix use of "/" and "\" in the XULRunnerpath, SWT browser is not able to find the XULRunner runtime. A simple test would be to create SWT browser instance with SWT.MOZILLA. This will lead to an exception in XPCOM. Suggested patch provided. Propose as an enhancement request. 

Reproducible: Always
Comment 1 wuzhe CLA 2010-12-19 21:47:46 EST
Created attachment 185514 [details]
patch

Patch.
Comment 2 Grant Gayed CLA 2010-12-20 12:38:50 EST
fixed > 20101220 (applied patch will be in next comment)
Comment 3 Grant Gayed CLA 2010-12-20 12:39:24 EST
Created attachment 185568 [details]
applied patch