Refactor desktop-detection logic to become an explicit propertyv5.15.0-beta2

The __isRootPanel internal property was used to detect whether the
keyboard is running in as a stand-alone, toplevel window in the desktop.
The value of the property depended on the location of the keyboard panel
within the QQuickItem hierarchy, which is fragile.

Instead, declare an explicit property desktopPanel that is set to true
by the desktop integration when the panel has been created.

Change-Id: Idb9e90e01e9b1c643e84287716e0b964256b115e
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>

1 files changed, 2 insertions, 4 deletions

diff --git a/src/virtualkeyboard/qvirtualkeyboardinputcontext_p.cpp b/src/virtualkeyboard/qvirtualkeyboardinputcontext_p.cpp
index 2c945356..a7c0aad9 100644
--- a/src/virtualkeyboard/qvirtualkeyboardinputcontext_p.cpp
+++ b/src/virtualkeyboard/qvirtualkeyboardinputcontext_p.cpp
@@ -273,15 +273,13 @@ void QVirtualKeyboardInputContextPrivate::onInputItemChanged()
     if (QObject *item = inputItem()) {
         if (QQuickItem *vkbPanel = qobject_cast<QQuickItem*>(inputPanel)) {
             if (QQuickItem *quickItem = qobject_cast<QQuickItem*>(item)) {
-                const QVariant isRootItem = vkbPanel->property("__isRootItem");
+                const QVariant isDesktopPanel = vkbPanel->property("desktopPanel");
                 /*
                     For integrated keyboards, make sure it's a sibling to the overlay. The
                     high z-order will make sure it gets events also during a modal session.
                 */
-                if (isRootItem.isValid() && !isRootItem.toBool()) {
+                if (isDesktopPanel.isValid() && !isDesktopPanel.toBool())
                     vkbPanel->setParentItem(quickItem->window()->contentItem());
-                    vkbPanel->setProperty("__reparented", true);
-                }
             }
         }
     } else {