Summary: | [ProblemsView] View content doesn't follow selection after hiding/unhiding view | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Andrey Loskutov <loskutov> | ||||
Component: | IDE | Assignee: | Dirk Fauth <dirk.fauth> | ||||
Status: | VERIFIED FIXED | QA Contact: | |||||
Severity: | major | ||||||
Priority: | P3 | CC: | bsd | ||||
Version: | 4.5 | Keywords: | greatfix | ||||
Target Milestone: | 4.5 M7 | ||||||
Hardware: | PC | ||||||
OS: | All | ||||||
See Also: |
https://bugs.eclipse.org/bugs/show_bug.cgi?id=457939 https://git.eclipse.org/r/45317 https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=3ad7a49a2fcb12fbfb35f3f31eb570186ab1203f |
||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Andrey Loskutov
2015-04-06 04:41:07 EDT
OK, found how to reproduce. Open few text editors in Eclipse. Restart and now steps are reproducible. The errors from bug 463957 are not appearing. The key thing seems to be opened but not initialized editor tabs, eventually side effect of bug 457939? Created attachment 252172 [details]
test workspace to reproduce
New Gerrit change created: https://git.eclipse.org/r/45317 Dirk, as far as I can see, bug 463957 is the same as this one and will be fixed by your patch too. If yes, I will make it duplicate. (In reply to Andrey Loskutov from comment #4) > Dirk, as far as I can see, bug 463957 is the same as this one and will be > fixed by your patch too. If yes, I will make it duplicate. Yes it looks similar. *** Bug 463957 has been marked as a duplicate of this bug. *** Gerrit change https://git.eclipse.org/r/45317 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=3ad7a49a2fcb12fbfb35f3f31eb570186ab1203f By way of background: MPlaceholders are a way to reference the same part instance from multiple perspectives (MPlaceholder#ref). They're necessary as the E3.x model had parts be shared across perspectives within a window (e.g., the Problems view on the Java perspective was the same Problems view as shown in the Debug view). Each part maintains a link back to its currently visible placeholder (MPart#curSharedRef). Dirk's fix in bug 457939 corrected an issue such that when the MPlaceholder was destroyed (e.g., due to a perspective being closed) its the corresponding part's curSharedRef is nulled out, if that MPlaceholder was the currently visible reference. It's perfectly valid for the curSharedRef to be null. For example, when the currently visible perspective isn't showing a view that is open in another perspective. But it rarely occurred previously. I'll review the rest of the callers of #getCurSharedRef() today. I'll add one other thing: there's a few small windows when MPart's curSharedRef will be out of sync with what's happening. A selection event, like what this code in PartServiceImpl was handling, is one of them. The other one I can think of is when a perspective is realized. I'll mark this as closed and continue any discussion in bug 457939. Tested with Eclipse 4.5.0 Integration Build: I20150428-0100 and it looks good. |