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

Bug 338575

Summary: [Cocoa] Thumbnail sync view broken in 3.7
Product: [Tools] GEF Reporter: <h1055071>
Component: GEF-Legacy GEF (MVC)Assignee: gef-inbox <gef-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: Andreas.Muelder, nyssen, Silenio_Quarti
Version: 3.7   
Target Milestone: 3.7.1 (Indigo) M7   
Hardware: Macintosh   
OS: Mac OS X   
Whiteboard:
Bug Depends on: 339132    
Bug Blocks:    

Description CLA 2011-03-01 14:08:10 EST
This is on Mac Cocoa using GEF 3.7 and Eclipse 3.7. Something screwy is going on with the thumbnail view.

Make a Logic Example and open up the Overview and show the thumbnail in that view ("Show overview" button). Initially the thumbnail is blank (which is wrong), then if you create and/or move objects in the Logic Editor the thumbnail is out of sync.

(It's OK on Windows 7 and Mac Carbon so it's a Cocoa thing).
Comment 1 CLA 2011-03-03 09:42:30 EST
Added [Cocoa] to title
Comment 2 Alexander Nyßen CLA 2011-03-03 16:43:00 EST
Seems to be an SWT regression we are facing here, because I cannot reproduce the problem with Eclipse 3.6.2 and the current GEF HEAD version, while I can with Eclipse  3.7M5.
Comment 3 Alexander Nyßen CLA 2011-03-03 17:09:46 EST
(In reply to comment #2)
> Seems to be an SWT regression we are facing here, because I cannot reproduce
> the problem with Eclipse 3.6.2 and the current GEF HEAD version, while I can
> with Eclipse  3.7M5.

It is also reproducible with 3.7M4 and current GEF HEAD.
Comment 4 Alexander Nyßen CLA 2011-03-03 17:29:18 EST
(In reply to comment #3)
> (In reply to comment #2)
> > Seems to be an SWT regression we are facing here, because I cannot reproduce
> > the problem with Eclipse 3.6.2 and the current GEF HEAD version, while I can
> > with Eclipse  3.7M5.
> 
> It is also reproducible with 3.7M4 and current GEF HEAD.

And not reproducible with 3.7M3, so we seem to be facing an SWT regression that has been introduced with 3.7M4.
Comment 5 CLA 2011-03-03 17:36:36 EST
Pity it's not in a Git repo, you could have done Git "Bisect" to find it. ;-) Thanks for this, Alexander I was going to trace it to a particular build tomorrow. Perhaps an SWT person might have an idea?
Comment 6 Alexander Nyßen CLA 2011-03-03 17:51:23 EST
BTW, it can also be reproduced using the ThumbnailExample of Draw2d:

http://dev.eclipse.org/viewcvs/viewvc.cgi/org.eclipse.gef/examples/org.eclipse.draw2d.examples/src/org/eclipse/draw2d/examples/thumbnail/?root=Tools_Project
Comment 7 CLA 2011-03-05 06:41:50 EST
What would be the best way to track this down? We might assume that a breaking change happened in SWT sometime between the release of 3.7M3 (released 28 Oct 2010) and 3.M4 (released 8 Dec 2010). I've looked through the commit messages of the SWT repo for likely candidates. But in order to home in on where the regression was introduced I would need to build SWT for that snapshot. I don't know how to do this. Is there a place where the integration builds are stored?
Comment 8 CLA 2011-04-07 09:56:25 EDT
The fix to Bug 339132 has fixed this issue.
Comment 9 Alexander Nyßen CLA 2011-04-07 10:14:31 EDT
Phil, thanks for verifying this! Resolving as fixed then.
Comment 10 Alexander Nyßen CLA 2011-11-18 10:46:54 EST
Reopening, because dependent bug #339132 changed state.
Comment 11 Alexander Nyßen CLA 2011-11-21 05:27:03 EST
Resolving as fixed again. As stated in bug #339132, this was a false alarm.