tmf: Execute TmfEventTable's request handler only on success
authorAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Fri, 14 Nov 2014 19:31:51 +0000 (14:31 -0500)
committerAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Mon, 1 Dec 2014 20:07:50 +0000 (15:07 -0500)
commit73fce654eb027eefa888a36d79097e6acf567fdc
tree1c43c4aa400f6925ae2d4abce317ad6f93e9f8b7
parent1786026ded6c8196bf863a766894d5d2778a8df2
tmf: Execute TmfEventTable's request handler only on success

The handler for the TmfTimeSynchSignal in TmfEventsTable sends an event
request to "position" the experiment to the correct timestamp/event. This
request executes a seek() on the experiment itself, and is located in the
handleCompleted() callback.

It should be moved to the handleSuccess() callback instead: this
repositioning of the experiment should only be done when the request
finishes succesfully. It should not be run if the request was cancelled,
for example because the executor was flooded with requests and cancelled
some of them.

Partial fix for bug 451615.

Change-Id: I905bb5d19aa0fe40f9d63bfd2e4f60d90aba3050
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/36517
Tested-by: Hudson CI
org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/viewers/events/TmfEventsTable.java
This page took 0.026366 seconds and 5 git commands to generate.