Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 345716 - Firefox 3.0.6 - NumberFormatExceptions and layout broken
Summary: Firefox 3.0.6 - NumberFormatExceptions and layout broken
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.4   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.4 RC1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-13 07:12 EDT by Sabine Neubauer CLA
Modified: 2011-05-31 08:42 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sabine Neubauer CLA 2011-05-13 07:12:36 EDT 
Build Identifier: 1.4M7

Using Firefox 3.0.6 is not possible anymore. Many NumberFormatExceptions like the appended one appear and the layout is totally broken.

2011-05-13 13:04:30.516:WARN:/:ERROR:  Failed to execute PhaseListener before phase 'PREPARE_UI_ROOT'.
java.lang.NumberFormatException: For input string: "NaN"
	at java.lang.NumberFormatException.forInputString(Unknown Source)
	at java.lang.Integer.parseInt(Unknown Source)
	at java.lang.Integer.parseInt(Unknown Source)
	at org.eclipse.rwt.internal.textsize.MeasurementOperator.getSize(MeasurementOperator.java:157)
	at org.eclipse.rwt.internal.textsize.MeasurementOperator.createProbeResult(MeasurementOperator.java:115)
	at org.eclipse.rwt.internal.textsize.MeasurementOperator.readMeasuredFontProbeSizes(MeasurementOperator.java:108)
	at org.eclipse.rwt.internal.textsize.MeasurementOperator.handleStartupProbeMeasurementResults(MeasurementOperator.java:52)
	at org.eclipse.rwt.internal.textsize.MeasurementListener.handleStartupProbeMeasurementResults(MeasurementListener.java:54)
	at org.eclipse.rwt.internal.textsize.MeasurementListener.beforePhase(MeasurementListener.java:41)
	at org.eclipse.rwt.internal.lifecycle.PhaseListenerManager.notifyBeforePhase(PhaseListenerManager.java:72)
	at org.eclipse.rwt.internal.lifecycle.RWTLifeCycle.continueLifeCycle(RWTLifeCycle.java:181)
	at org.eclipse.rwt.internal.lifecycle.RWTLifeCycle$UIThreadController.run(RWTLifeCycle.java:87)
	at java.lang.Thread.run(Unknown Source)
	at org.eclipse.rwt.internal.lifecycle.UIThread.run(UIThread.java:102)

Reproducible: Always

Steps to Reproduce:
1. start control demo
2. use firefox 3.0.6 to access the webinterface
=> many exceptions in log and broken layout
Comment 1 Ivan Furnadjiev CLA 2011-05-13 09:25:39 EDT 
To resolve bug 264448, bug 340841 and bug 340841 we are using getBoundingClientRect (width/height) in Label.js#_computeObjectNeededDimensions and MultiCellWidget.js# _computeTextDimensions (Firefox only). In FF 3.0.x boundingClientRect width/height properties are missing. The fix checks if width/height properties exist on boundingClientRect before we are using them, otherwise use scrollWidth/scrollHeight as before. Changes are in CVS HEAD.
Comment 2 Sabine Neubauer CLA 2011-05-31 08:42:06 EDT 
Today I have updated to RAP 1.4 RC2. Using FF 3.0.6 for our RAP application is no problem anymore.
=> bug fix verfied with 1.4 RC2