Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 351849 - [Table] JavaScript Error in context of cell tooltips
Summary: [Table] JavaScript Error in context of cell tooltips
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.5 M1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard: sr141
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-12 10:14 EDT by Sabine Neubauer CLA
Modified: 2011-07-26 04:13 EDT (History)
0 users

See Also:


Attachments
Used client.js (884.05 KB, application/x-javascript)
2011-07-12 10:18 EDT, Sabine Neubauer CLA
no flags Details
Proposed patch for v14_Maintenance branch (2.34 KB, patch)
2011-07-22 03:47 EDT, Ivan Furnadjiev CLA
ivan: review?
rsternberg: review+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sabine Neubauer CLA 2011-07-12 10:14:30 EDT
Build Identifier: 

In our applications we have several tables with cell tooltips. Sometimes a javascript error occurs in the method Table.js#_onClientAreaMouseMove. Unfortunately, I am not able to reproduce the problem properly. Hence, I can currently not provide a small snippet.

The error message on IE is the following:
User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0;
SLCC2; .NET
CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0;
InfoPath.3;
MS-RTC LM 8; .NET4.0C; .NET4.0E)
Timestamp: Wed, 6 Jul 2011 14:18:01 UTC

Message: 'null' is null or not an object
Line: 170
Char: 32390
Code: 0
URI: ...

Further information:
- RAP 1.4 RC2
- used browsers: InternetExplorer 8 and Firefox (3.6 or newer)

Reproducible: Sometimes
Comment 1 Sabine Neubauer CLA 2011-07-12 10:18:31 EDT
Created attachment 199500 [details]
Used client.js
Comment 2 Ivan Furnadjiev CLA 2011-07-13 02:11:10 EDT
Thanks for the new bug report, Sabine. Current information does not help us much to isolate the problem. Could you describe some steps that provoke the error in your application - is Table in popup/dialog/wizard, is this happen when disposing the table? Please run your application in Fifrefox and provide the complete error information from the error page.
Comment 3 Sabine Neubauer CLA 2011-07-15 06:28:38 EDT
The only information I have at the moment:
The table is part of a view containing several sections which are always expanded.

Currently, I am not able to reproduce the bug. I don't think that's because it now works but I need further information from the end user who reported the problem to me. As soon I have further information I will let you know. Thanks in advance.
Comment 4 Sabine Neubauer CLA 2011-07-22 03:01:24 EDT
I am now able to reproduce the problem very reliable.

1. Starts the RAP examples demo
2. Open your Internet Explorer and set the zoom factor to 110%
3. Now open the examples demo (it is important that the zoom factor is set before first access!)
4. Open the TableViewer "tab"
5. Move the mouse over the table in order to get cell tooltips
=> Java script error
Comment 5 Ivan Furnadjiev CLA 2011-07-22 03:13:28 EDT
Thanks Sabine for the hint. I got the JavaScript error in IE9 with zoom level 110%.
Comment 6 Ivan Furnadjiev CLA 2011-07-22 03:47:18 EDT
Created attachment 200152 [details]
Proposed patch for v14_Maintenance branch

This bug does not exist in CVS HEAD. The error appears when the last visible item is partly visible and you request a cell tooltip for it. The method Table.js#_getRowAtPoint returns null in this case. The patch checks if row is not null in Table.js#_onClientAreaMouseMove.
Comment 7 Sabine Neubauer CLA 2011-07-22 04:56:31 EDT
(In reply to comment #6)
> Created attachment 200152 [details]
> Proposed patch for v14_Maintenance branch
> 
> This bug does not exist in CVS HEAD. The error appears when the last visible
> item is partly visible and you request a cell tooltip for it. The method
> Table.js#_getRowAtPoint returns null in this case. The patch checks if row is
> not null in Table.js#_onClientAreaMouseMove.

Thanks for the patch it solves the problem. But I think the reason for the error can also be another one. Because the problem occurs in our application also if the table has only one or two rows which are completely visible and the table has no scrollbar in this case. Precondition is to use a zoomed Internet Explorer.
Comment 8 Ivan Furnadjiev CLA 2011-07-26 04:13:37 EDT
Applied patch to v14_Maintenance branch.