Bugzilla – Full Text Bug Listing
| Summary: | NPE in editor mouse move handler | ||
|---|---|---|---|
| Product: | [ECD] Orion | Reporter: | Grant Gayed <grant_gayed> |
| Component: | Editor | Assignee: | Project Inbox <orion.editor-inbox> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | emoffatt |
| Version: | unspecified | ||
| Target Milestone: | 8.0 | ||
| Hardware: | PC | ||
| OS: | Mac OS X | ||
| Whiteboard: | |||
I've added a check for 'self._listener' to account for situations where the state may have changed before the event fires... http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=12d00ba8f5f692f57b758b44da5e0aa12f6fae70 We likely also want to 'clear' the timeout but I'm not sure where the code should go... |
Our logging of uncaught errors has logged many NPE's at editor/editor.js:828: if (!tooltip.OKToHover(self._listener.lastMouseX, self._listener.lastMouseY)) { return; } It looks like "self._listener" can be null when this line is hit. Blame says that this line was introduced as part of the work for bug 429250.