From be6595c6d5a7624ed724d36a0857227ad55b1372 Mon Sep 17 00:00:00 2001 From: Vladislav Eliseev Date: Mon, 12 Sep 2016 12:27:43 +0300 Subject: [PATCH] SRADA-1040: fix bug with missing Processes chart * Due to wrong synchronization Processes chart is not created second time. * Bug related to the FindBugs MT_CORRECTNESS warning. * Bug introduced in SRADA-703, commit ea27fce. Change-Id: I87ba2ea3785e601b2e72782ffb00a4b54a5fe774 --- .../tizen/dynamicanalyzer/ui/timeline/chart/CPUChart.java | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/org.tizen.dynamicanalyzer/src/org/tizen/dynamicanalyzer/ui/timeline/chart/CPUChart.java b/org.tizen.dynamicanalyzer/src/org/tizen/dynamicanalyzer/ui/timeline/chart/CPUChart.java index c5869ff..c5fb84c 100755 --- a/org.tizen.dynamicanalyzer/src/org/tizen/dynamicanalyzer/ui/timeline/chart/CPUChart.java +++ b/org.tizen.dynamicanalyzer/src/org/tizen/dynamicanalyzer/ui/timeline/chart/CPUChart.java @@ -78,8 +78,6 @@ public class CPUChart extends TimelineChart { private DAChartBoardItem item = null; private DAChartBoardHeightChangeableItem processItem = null; private Map childSeriesMap; - - private Boolean isProcessChart = false; public CPUChart(String pageID) { super(pageID); @@ -148,14 +146,9 @@ public class CPUChart extends TimelineChart { Display.getDefault().syncExec(new Runnable() { @Override public void run() { - synchronized (isProcessChart) { - if(isProcessChart == false) { - isProcessChart = true; - } - else { - return; - } - } + if (processItem != null) + return; + processItem = new DAChartBoardHeightChangeableItem(item, "Processes"); processItem.useExpand(false); -- 2.7.4