commit d9cf47449e88032803aa4fa3429cd607a074d5ad
author Justin Luth <justin_luth@sil.org> Fri Oct 16 20:41:12 2015 +0300
committer Justin Luth <justin_luth@sil.org> Sat Oct 17 11:55:31 2015 +0000
tree 881dc7471480dcd17204c6d14de6c0430014fbc3
parent a9b31303b76326e0dc0043082082c5fc1dc69be9

tdf#95096 avoid invalid cursor position exception

Somehow, accessible text can lose focus and so when
attempting to set the cursor, the string we were just
working with may no longer be available, so verify that
the cursor position is still valid before setting it.

If the text can just disappear like that, then potentially
we are still forcing the cursor into the wrong place
(since the string is different than expected).  However
we can at least make sure the program doesn't
crash/exit without saving...

Change-Id: I8dcaf4625d187cb75008bdc0af0e3a546090a018
Reviewed-on: https://gerrit.libreoffice.org/19414
Reviewed-by: Justin Luth <justin_luth@sil.org>
Tested-by: Justin Luth <justin_luth@sil.org>

vcl/unx/gtk/window/gtksalframe.cxx

diff --git a/vcl/unx/gtk/window/gtksalframe.cxx b/vcl/unx/gtk/window/gtksalframe.cxx
index ebc8516..25f3fe5 100644
--- a/vcl/unx/gtk/window/gtksalframe.cxx
+++ b/vcl/unx/gtk/window/gtksalframe.cxx
@@ -4712,9 +4712,12 @@ gboolean GtkSalFrame::IMHandler::signalIMDeleteSurrounding( GtkIMContext*, gint 
        if (nDeletePos < nPosition)
        {
            if (nDeleteEnd <= nPosition)
                xText->setCaretPosition( nPosition-(nDeleteEnd-nDeletePos) );
                nPosition = nPosition - (nDeleteEnd - nDeletePos);
            else
                xText->setCaretPosition( nDeletePos );
                nPosition = nDeletePos;

            if (xText->getCharacterCount() >= nPosition)
                xText->setCaretPosition( nPosition );
        }
        return true;
    }