497232: Managing model notifications in UI thred

Change-Id: Ib31411aa1edca16d0f1c67a2c8b37c0d2a13459a Task-Url: https://bugs.eclipse.org/bugs/show_bug.cgi?id=497232 Signed-off-by: Francesco Guidieri <francesco.guidieri@gmail.com>
eclipse-emf-parsley · Jul 5, 2016 · ada7d24 · ada7d24
1 parent c7c269e
commit ada7d24
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/...eclipse.emf.parsley.common/src/org/eclipse/emf/parsley/composite/FormDetailComposite.java b/...eclipse.emf.parsley.common/src/org/eclipse/emf/parsley/composite/FormDetailComposite.java
@@ -111,7 +111,12 @@ public void dispose() {
 		@Override
 		public void notifyChanged(Notification msg) {
 			if (!disposing) {
-				updateTitle(model);
+				getDisplay().syncExec(new Runnable() {
+					@Override
+					public void run() {
+						updateTitle(model);
+					}
+				});
 			}
 		}
 	}

diff --git a/...eclipse.emf.parsley.tests/src/org/eclipse/emf/parsley/tests/FormDetailCompositeTest.xtend b/...eclipse.emf.parsley.tests/src/org/eclipse/emf/parsley/tests/FormDetailCompositeTest.xtend
@@ -60,7 +60,9 @@ class FormDetailCompositeTest extends AbstractEmfParsleyControlBasedTest {
 		formDetailComposite.init(o, editingDomain)
 		"Class With Name Test".assertEquals(formDetailComposite.scrolledForm.text)
 		o.name = "Changed"
-		"Class With Name Changed".assertEquals(formDetailComposite.scrolledForm.text)
+		syncExecVoid[
+			"Class With Name Changed".assertEquals(formDetailComposite.scrolledForm.text)
+		]
 		formDetailComposite.dispose
 	}
 
@@ -69,7 +71,9 @@ class FormDetailCompositeTest extends AbstractEmfParsleyControlBasedTest {
 		formDetailComposite.init(o)
 		"Class With Name Test".assertEquals(formDetailComposite.scrolledForm.text)
 		o.name = "Changed"
-		"Class With Name Changed".assertEquals(formDetailComposite.scrolledForm.text)
+		syncExecVoid[
+			"Class With Name Changed".assertEquals(formDetailComposite.scrolledForm.text)
+		]
 		formDetailComposite.dispose
 	}
 }