diff options
author | Justin Luth <justin_luth@sil.org> | 2017-02-20 10:35:25 +0300 |
---|---|---|
committer | Takeshi Abe <tabe@fixedpoint.jp> | 2017-02-20 17:00:12 +0000 |
commit | 1c3a784d38fc5986c78bab8d283c4661347baf43 (patch) | |
tree | 618dcc974450677584f00b9c8cb61117cc126bad | |
parent | 6fbce0cb340448dc0074ff2462babc657da46056 (diff) |
tdf#105256 gtk findfocus return only if focused
A patch related to tdf#96685 (commit 598e6a024163f1510d076000788b7745625f5ed5)
caused the initial regression that Japanese-mozc would freeze(tdf#100903).
That patch only modified what happened when the text was marked as focused.
However, the regression fix for Japanese bug 100903 (commit
1c81af2c1814e8bd12701f85e09cebf5fe206647) was not placed
inside of the "if focused" code, and so it applies to more situations
than were broken by the first regression, thus causing new bug 105256.
So, applying fix 100903 ONLY when focused solves both problems.
Change-Id: I1d8381f2d8cf793d481287ae975ce8a485b6c573
Reviewed-on: https://gerrit.libreoffice.org/34453
Reviewed-by: Justin Luth <justin_luth@sil.org>
Tested-by: Justin Luth <justin_luth@sil.org>
Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
-rw-r--r-- | vcl/unx/gtk/gtksalframe.cxx | 4 | ||||
-rw-r--r-- | vcl/unx/gtk3/gtk3gtkframe.cxx | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/vcl/unx/gtk/gtksalframe.cxx b/vcl/unx/gtk/gtksalframe.cxx index 7ec919744272..1f41463389fb 100644 --- a/vcl/unx/gtk/gtksalframe.cxx +++ b/vcl/unx/gtk/gtksalframe.cxx @@ -3779,9 +3779,9 @@ uno::Reference<accessibility::XAccessibleEditableText> uno::Reference<accessibility::XAccessibleEditableText>(xContext, uno::UNO_QUERY); if (xText.is()) return xText; + if (xState->contains(accessibility::AccessibleStateType::MANAGES_DESCENDANTS)) + return uno::Reference< accessibility::XAccessibleEditableText >(); } - if (xState->contains(accessibility::AccessibleStateType::MANAGES_DESCENDANTS)) - return uno::Reference< accessibility::XAccessibleEditableText >(); } for (sal_Int32 i = 0; i < xContext->getAccessibleChildCount(); ++i) diff --git a/vcl/unx/gtk3/gtk3gtkframe.cxx b/vcl/unx/gtk3/gtk3gtkframe.cxx index e2d19fe6ffb8..26ffb0a3769d 100644 --- a/vcl/unx/gtk3/gtk3gtkframe.cxx +++ b/vcl/unx/gtk3/gtk3gtkframe.cxx @@ -3983,9 +3983,9 @@ uno::Reference<accessibility::XAccessibleEditableText> uno::Reference<accessibility::XAccessibleEditableText>(xContext, uno::UNO_QUERY); if (xText.is()) return xText; + if (xState->contains(accessibility::AccessibleStateType::MANAGES_DESCENDANTS)) + return uno::Reference< accessibility::XAccessibleEditableText >(); } - if (xState->contains(accessibility::AccessibleStateType::MANAGES_DESCENDANTS)) - return uno::Reference< accessibility::XAccessibleEditableText >(); } for (sal_Int32 i = 0; i < xContext->getAccessibleChildCount(); ++i) |