summaryrefslogtreecommitdiffstats
path: root/src/opengl/qopenglframebufferobject.cpp
diff options
context:
space:
mode:
authorTinja Paavoseppä <tinja.paavoseppa@qt.io>2024-01-30 10:13:42 +0200
committerTinja Paavoseppä <tinja.paavoseppa@qt.io>2024-01-30 17:41:48 +0000
commit8df46c8890a843a651c86d198540d6318dbaa9a1 (patch)
tree3cba3fa8e9ce47639582d0d190f8ee8cc9d4de38 /src/opengl/qopenglframebufferobject.cpp
parent8feb41e4462c9879a831d1c753d4200c3287e557 (diff)
Android: Create QtWindow layout and set parent in the calling thread
In the QtWindow constructor, creation of the layout and setting the parent window were posted on the Android UI thread, leading to them being called at a later point. If QAndroidPlatformWindow did not have a parent at the point it was created in, but setParent() was called shortly after, the QtWindow.setParent() call with the actual intended parent got invoked before the Runnable posted in the constructor got ran, leading to the parent being overwritten with the null one passed to the constructor, essentially leaving the QtWindow as a top level one, while the QAndroidPlatformWindow was a child window. The above would happen more often with foreign child windows, sometimes causing hang ups when the parent of the foreign child window was shown. Creating the QtLayout outside of the Android UI thread seems to be safe, as long as we only modify its view hierarchy inside it. Pick-to: 6.7 Task-number: QTBUG-116187 Change-Id: If1ed1983f5d6ba56e625148ee6a61771234a2aa1 Reviewed-by: Assam Boudjelthia <assam.boudjelthia@qt.io>
Diffstat (limited to 'src/opengl/qopenglframebufferobject.cpp')
0 files changed, 0 insertions, 0 deletions