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

Bug 62200

Summary: Only persist DisplayView contents on view closure
Product: [Eclipse Project] Platform Reporter: Jared Burns <jared_burns>
Component: DebugAssignee: Darin Wright <darin.eclipse>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: darin.eclipse
Version: 3.0   
Target Milestone: 3.1   
Hardware: PC   
OS: Linux   
Whiteboard:
Bug Depends on: 65606    
Bug Blocks:    
Attachments:
Description Flags
DisplayView.62200.patch none

Description Jared Burns CLA 2004-05-13 19:50:53 EDT
When Bug 60039 is fixed, DisplayView#partHidden(...) should only save its state when the view is 
closed for the last time. The code to do this is already in the file; it's just commented out.
Comment 1 Jared Burns CLA 2004-06-03 15:27:41 EDT
Bug 60039 was fixed, but there's a problem with the implementation (bug 65606) that prevents us 
from using the feature.
Comment 2 Jared Burns CLA 2004-06-03 15:30:15 EDT
Created attachment 11562 [details]
DisplayView.62200.patch

This patch implements the change from our end. Will release once the UI bug is
fixed.
Comment 3 Jared Burns CLA 2004-06-14 19:41:09 EDT
Now that bug 65606 has been fixed, the attached patch functions correctly and can be released for 
RC3.
Comment 4 Darin Wright CLA 2004-06-15 09:59:50 EDT
Jared, although this is a "nice to fix" bug, I see no reason that this has to 
be fixed for 3.0. There is no change in behavior to the user, and no user 
information is lost without the fix. Is this correct?
Comment 5 Darin Wright CLA 2004-06-15 10:32:05 EDT
Removing RC3 target, as I don't believe this bug is critical.
Comment 6 Jared Burns CLA 2004-06-15 11:55:29 EDT
The code in HEAD was released last month with the knowledge that it was wrong.
We've just been waiting for bug 65606 to get fixed. I don't like the idea of
knowingly shipping the wrong code when we have the right code available, but the
performance problem here isn't critical.
Comment 7 Darin Wright CLA 2004-06-17 11:50:20 EDT
Deferred.
Comment 8 Darin Wright CLA 2004-08-03 14:25:16 EDT
open for 3.1
Comment 9 Jared Burns CLA 2004-08-10 17:00:14 EDT
Fixed in DisplayView. Please verify, DW.
Comment 10 Darin Wright CLA 2004-08-20 14:57:16 EDT
Verified