diff --git a/src/gui/kernel/qplatformwindow.h b/src/gui/kernel/qplatformwindow.h index b6e96a457e..6e2d823c4a 100644 --- a/src/gui/kernel/qplatformwindow.h +++ b/src/gui/kernel/qplatformwindow.h @@ -32,6 +32,7 @@ class QScreen; class QWindow; class QIcon; class QRegion; +class QPlatformBackingStore; class Q_GUI_EXPORT QPlatformWindow : public QPlatformSurface { @@ -117,6 +118,7 @@ public: virtual void requestUpdate(); bool hasPendingUpdateRequest() const; virtual void deliverUpdateRequest(); + virtual void setBackingStore(QPlatformBackingStore *) {} // Window property accessors. Platform plugins should use these // instead of accessing QWindow directly. diff --git a/src/platformsupport/fbconvenience/qfbscreen.cpp b/src/platformsupport/fbconvenience/qfbscreen.cpp index e804488005..a5d87fec04 100644 --- a/src/platformsupport/fbconvenience/qfbscreen.cpp +++ b/src/platformsupport/fbconvenience/qfbscreen.cpp @@ -47,6 +47,7 @@ bool QFbScreen::event(QEvent *event) void QFbScreen::addWindow(QFbWindow *window) { + Q_ASSERT(window->backingStore()); mWindowStack.prepend(window); if (!mPendingBackingStores.isEmpty()) { //check if we have a backing store for this window diff --git a/src/platformsupport/fbconvenience/qfbwindow.cpp b/src/platformsupport/fbconvenience/qfbwindow.cpp index 54716cc497..8df50a14da 100644 --- a/src/platformsupport/fbconvenience/qfbwindow.cpp +++ b/src/platformsupport/fbconvenience/qfbwindow.cpp @@ -3,9 +3,11 @@ #include "qfbwindow_p.h" #include "qfbscreen_p.h" +#include "qfbbackingstore_p.h" #include #include +#include QT_BEGIN_NAMESPACE @@ -111,5 +113,11 @@ void QFbWindow::repaint(const QRegion ®ion) for (auto rect : region) platformScreen()->setDirty(rect.translated(topLeft)); } - +void QFbWindow::setBackingStore(QPlatformBackingStore *store) +{ + Q_ASSERT(store); + Q_ASSERT_X(dynamic_cast(store), __FUNCTION__, + "Argument is not a QFbBackingStore."); + mBackingStore = static_cast(store); +} QT_END_NAMESPACE diff --git a/src/platformsupport/fbconvenience/qfbwindow_p.h b/src/platformsupport/fbconvenience/qfbwindow_p.h index 8367a10bce..8848e9dcf8 100644 --- a/src/platformsupport/fbconvenience/qfbwindow_p.h +++ b/src/platformsupport/fbconvenience/qfbwindow_p.h @@ -41,7 +41,7 @@ public: WId winId() const override { return mWindowId; } - void setBackingStore(QFbBackingStore *store) { mBackingStore = store; } + void setBackingStore(QPlatformBackingStore *store) override; QFbBackingStore *backingStore() const { return mBackingStore; } QFbScreen *platformScreen() const; diff --git a/src/plugins/platforms/android/qandroidplatformwindow.cpp b/src/plugins/platforms/android/qandroidplatformwindow.cpp index b1eba17d04..1bc4a04b98 100644 --- a/src/plugins/platforms/android/qandroidplatformwindow.cpp +++ b/src/plugins/platforms/android/qandroidplatformwindow.cpp @@ -5,9 +5,9 @@ #include "qandroidplatformwindow.h" #include "qandroidplatformopenglcontext.h" #include "qandroidplatformscreen.h" - +#include "qandroidplatformbackingstore.h" #include "androidjnimain.h" - +#include "qpa/qplatformbackingstore.h" #include #include #include @@ -169,4 +169,12 @@ void QAndroidPlatformWindow::applicationStateChanged(Qt::ApplicationState) QWindowSystemInterface::flushWindowSystemEvents(); } +void QAndroidPlatformWindow::setBackingStore(QPlatformBackingStore *store) +{ + Q_ASSERT(store); + Q_ASSERT_X(dynamic_cast(store), __FUNCTION__, + "Argument is not a QAndroidPlatformBackingStore."); + m_backingStore = static_cast(store); +} + QT_END_NAMESPACE diff --git a/src/plugins/platforms/android/qandroidplatformwindow.h b/src/plugins/platforms/android/qandroidplatformwindow.h index 6fccc2e7fe..ee2771ee11 100644 --- a/src/plugins/platforms/android/qandroidplatformwindow.h +++ b/src/plugins/platforms/android/qandroidplatformwindow.h @@ -50,7 +50,7 @@ public: virtual void applicationStateChanged(Qt::ApplicationState); - void setBackingStore(QAndroidPlatformBackingStore *store) { m_backingStore = store; } + void setBackingStore(QPlatformBackingStore *store) override; QAndroidPlatformBackingStore *backingStore() const { return m_backingStore; } virtual void repaint(const QRegion &) { } diff --git a/src/widgets/kernel/qwidget.cpp b/src/widgets/kernel/qwidget.cpp index b48cbf9f55..64e55bd34b 100644 --- a/src/widgets/kernel/qwidget.cpp +++ b/src/widgets/kernel/qwidget.cpp @@ -1329,15 +1329,14 @@ void QWidgetPrivate::create() } #endif - // Android doesn't allow to re-use the backing store. - // => force creation of a new one. -#ifdef Q_OS_ANDROID - QBackingStore *store = nullptr; -#else QBackingStore *store = q->backingStore(); -#endif usesRhiFlush = false; + // Re-use backing store, in case a new platform window was created and doesn't know about it. + // (e.g. QAndroidPlatformWindow) + if (store && q->windowHandle()) + q->windowHandle()->handle()->setBackingStore(store->handle()); + if (!store) { if (q->windowType() != Qt::Desktop) { if (q->isWindow()) {