Bug 244806 - [tml][vncviewer][bug] SWTRemoteDisplay state is not consistent on errors
Summary: [tml][vncviewer][bug] SWTRemoteDisplay state is not consistent on errors
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Sequoyah (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Fabio Fantato CLA
QA Contact:
URL:
Whiteboard: released_0.1
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-08-21 08:40 EDT by Fabio Rigo CLA
Modified: 2016-11-08 14:45 EST (History)
3 users (show)

See Also:


Attachments
Fix to the SWTRemoteDisplay state bug (1.10 KB, patch)
2008-09-02 16:58 EDT, Fabio Rigo CLA
fabio.fantato: iplog+
fabio.fantato: review+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fabio Rigo CLA 2008-08-21 08:40:37 EDT
Build ID: I20080617-2000

Steps To Reproduce:
During screen refresh, if an error occurs and the exception block is executed, the isActive flag is not updated. Therefore, it is not possible for an external component to check for the screen state. 

More information:
Comment 1 Fabio Rigo CLA 2008-09-02 16:58:31 EDT
Created attachment 111524 [details]
Fix to the SWTRemoteDisplay state bug
Comment 2 Fabio Rigo CLA 2008-09-02 17:01:28 EDT
I have posted an attachment that solves the issue. It simply updates the SWT remote display state so that external classes have accurate results when querying for screen status. 

This is a note to clarify the licensing issues:
I wrote the code of all new classes that are contained in the archive. I has
the right to contribute the code to Eclipse.
Comment 3 Fabio Rigo CLA 2008-09-03 09:40:01 EDT
One additional note: 
I, Fabio Rigo, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. I am authorized by my employer to make this contribution under the EPL.
Comment 4 Fabio Fantato CLA 2008-09-03 10:17:48 EDT
Code verified and patch applied
Comment 5 Fabio Fantato CLA 2008-09-05 13:33:03 EDT
integrated to version 0.1
Comment 6 Fabio Fantato CLA 2008-11-13 08:18:26 EST
CVS branch for 0.1 release: branch_dsdp_tml_official_0_1_0_R