Bug 502833 - Dragging bendpoints/endpoints crashes Eclipse
Summary: Dragging bendpoints/endpoints crashes Eclipse
Status: NEW
Alias: None
Product: GEF
Classification: Tools
Component: GEF-Legacy Draw2d (show other bugs)
Version: 3.11.0   Edit
Hardware: PC Windows 10
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: gef-inbox CLA
QA Contact: Patrick Tessier CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-09-30 05:11 EDT by Mickael ADAM CLA
Modified: 2017-06-19 06:53 EDT (History)
5 users (show)

See Also:

Attachments
Tree Layout crash (195.04 KB, image/gif)
2016-09-30 05:11 EDT, Mickael ADAM CLA 		no flags Details
Proposal fix - Git patch (1.08 KB, patch)
2017-04-21 04:48 EDT, Fanch Bonnabesse CLA 		no flags Details | Diff
View All 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 Mickael ADAM CLA 2016-09-30 05:11:54 EDT 
Created attachment 264505 [details]
Tree Layout crash

As report at bug 502165, in some case when we move a bendpoint from an edge. The edge is root with tree without closest distance.

use case:

 - Create a new ecore diagram using the file > new.
 - Create 3 classes 1, 2 and 3.
 - Drag a bendpoint
=> Eclipse is crashed (see attachment)

This seems to come from the validation of the layout. The figure figure LightweightSystem$Rootfigure is constantly invalidate since the scrollbar is growing.

This bug occurs in oxygen and can be reproduced in mars/neon/oxygen version of Papyrus(which use GMF).
Comment 1 Fanch Bonnabesse CLA 2017-04-07 10:30:51 EDT 
Modification of the product (GMF to GEF) because the bug is contained on the class "org.eclipse.draw2d.DeferredUpdateManager.class".
Comment 2 Fanch Bonnabesse CLA 2017-04-21 04:48:29 EDT 
Created attachment 267900 [details]
Proposal fix - Git patch
Comment 3 Alexander Nyßen CLA 2017-04-21 05:18:41 EDT 
Is this fix critical or is there a potential workaround in Papyrus?
Comment 4 Alexander Nyßen CLA 2017-05-01 10:42:36 EDT 
Fanch, the fix does not seem appropriate to me. What is the 'motivation' behind it. What do you think is the root cause?
Comment 5 Patrick Tessier CLA 2017-05-03 09:56:04 EDT 
Hi, 
This is a critical bug for papyrus. 
I let Fanch when I come back for  holidays to explain the fix.
Comment 6 Alexander Nyßen CLA 2017-05-03 10:10:51 EDT 
We will only accept patches up to M7, so we should hurry up on this one.