[[ Bug 22914 ]] Call doProcess in updateKeyboardVisible

montegoulding · montegoulding · commit 16a810731fb8 · 2020-09-22T13:45:58.000+10:00
This patch adds a call to `doProcess` to `updateKeyboardVisible` so that we
yield to the engine thread after sending the `keyboardActivated` or
`keyboardDeactivated` messages. This ensures the yield to android thread in
`wait` is broken.
diff --git a/docs/notes/bugfix-22914.md b/docs/notes/bugfix-22914.md
@@ -0,0 +1 @@
+# Fix `keyboardActivated` and `keyboardDeactivated` not breaking `wait for messages` on Android.
diff --git a/engine/src/java/com/runrev/android/Engine.java b/engine/src/java/com/runrev/android/Engine.java
@@ -1557,6 +1557,10 @@ void updateKeyboardVisible()
 			doKeyboardHidden();
 		
 		m_keyboard_sizechange = true;
+
+		// Make sure we trigger handling
+		if (m_wake_on_event)
+			doProcess(false);
 	}
 	
 	void updateOrientation(int p_orientation)

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+# Fix `keyboardActivated` and `keyboardDeactivated` not breaking `wait for messages` on Android.
Original file line number	Diff line number	Diff line change
`@@ -1557,6 +1557,10 @@ void updateKeyboardVisible()`
`1557`	`1557`	`doKeyboardHidden();`
`1558`	`1558`
`1559`	`1559`	`m_keyboard_sizechange = true;`
	`1560`	`+`
	`1561`	`+ // Make sure we trigger handling`
	`1562`	`+ if (m_wake_on_event)`
	`1563`	`+ doProcess(false);`
`1560`	`1564`	`}`
`1561`	`1565`
`1562`	`1566`	`void updateOrientation(int p_orientation)`