Merge branch 'release/v2.4.x' into develop

Author: andydotxyz

widget/entry.go

@@ -953,10 +953,11 @@ func (e *Entry) cutToClipboard(clipboard fyne.Clipboard) {
 	e.copyToClipboard(clipboard)
 	e.setFieldsAndRefresh(e.eraseSelection)
 	e.propertyLock.RLock()
+	content := e.Text
 	cb := e.OnChanged
 	e.propertyLock.RUnlock()
 	if cb != nil {
-		cb(e.Text)
+		cb(content)
 	}
 	e.Validate()
 }
@@ -1221,10 +1222,11 @@ func (e *Entry) selectingKeyHandler(key *fyne.KeyEvent) bool {
 		// clears the selection -- return handled
 		e.setFieldsAndRefresh(e.eraseSelection)
 		e.propertyLock.RLock()
+		content := e.Text
 		cb := e.OnChanged
 		e.propertyLock.RUnlock()
 		if cb != nil {
-			cb(e.Text)
+			cb(content)
 		}
 		e.Validate()
 		return true

widget/entry_internal_test.go

@@ -287,6 +287,21 @@ func TestEntry_EraseSelection(t *testing.T) {
 	assert.Equal(t, -1, b)
 }
 
+func TestEntry_CallbackLocking(t *testing.T) {
+	e := &Entry{}
+	called := 0
+	e.OnChanged = func(_ string) {
+		e.propertyLock.Lock()
+		called++ // Just to not have an empty critical section.
+		e.propertyLock.Unlock()
+	}
+
+	test.Type(e, "abc123")
+	e.selectAll()
+	e.TypedKey(&fyne.KeyEvent{Name: fyne.KeyBackspace})
+	assert.Equal(t, 7, called)
+}
+
 func TestEntry_MouseClickAndDragOutsideText(t *testing.T) {
 	entry := NewEntry()
 	entry.SetText("A\nB\n")