Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 357396 - [LTTng/TMF] TmfTimeSynchSignal not handled properly outside GUI thread
Summary: [LTTng/TMF] TmfTimeSynchSignal not handled properly outside GUI thread
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: LinuxTools (show other bugs)
Version: unspecified   Edit
Hardware: All Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Bernd Hufmann CLA
QA Contact: Francois Chouinard CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-12 11:43 EDT by Bernd Hufmann CLA
Modified: 2022-01-13 14:52 EST (History)
1 user (show)

See Also:


Attachments
Patch for bug (8.63 KB, patch)
2011-09-13 11:00 EDT, Bernd Hufmann CLA
jjohnstn: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Bernd Hufmann CLA 2011-09-12 11:43:15 EDT
The Control Flow View and Resource View don't handle the TmfTimeSynchSignal if it was sent outside the GUI thread. 

Steps to reproduce.

1) Load LTTng trace
2) Have Control Flow View and Resource View open
3) Click with mouse into small Histogram View. Then the current time isn't selected in neither the Control Flow View and Resource View.
Comment 1 Bernd Hufmann CLA 2011-09-12 15:58:42 EDT
The same problem happens in the TMF Sequence Diagram View (loader class).
Comment 2 Bernd Hufmann CLA 2011-09-13 11:00:19 EDT
Created attachment 203253 [details]
Patch for bug

This patch fixes the described bug. When receiving the signal TmfTimeSyncSignal the relevant components check if it they are running in the GUI thread or not. If they are already in the GUI thread, the update is done immediately. If not in GUI thread, then the update is queued to be executed in the GUI thread.
Comment 3 Bernd Hufmann CLA 2011-09-13 11:03:03 EDT
Comment on attachment 203253 [details]
Patch for bug

Patch committed!
Comment 4 Bernd Hufmann CLA 2011-09-13 11:04:36 EDT
Bug fixed after committing changes of patch.
Comment 5 Francois Chouinard CLA 2012-03-26 13:52:18 EDT
Delivered in 0.9.0