wasm: don’t leak QWasmScreen member variables

m_compositor and m_eventTranslator are owned by the
screen object and can be held in a unique_ptr.

Pick-to: 6.2
Change-Id: I7d5e19e7ac6f762f360d6af0fc9500e6964f737e
Reviewed-by: Morten Johan Sørvig <[email protected]>

1 files changed, 4 insertions, 4 deletions

diff --git a/src/plugins/platforms/wasm/qwasmscreen.cpp b/src/plugins/platforms/wasm/qwasmscreen.cpp
index 7a8bb4f6867..0b11415fd3a 100644
--- a/src/plugins/platforms/wasm/qwasmscreen.cpp
+++ b/src/plugins/platforms/wasm/qwasmscreen.cpp
@@ -54,9 +54,9 @@ const char * QWasmScreen::m_canvasResizeObserverCallbackContextPropertyName = "d
 
 QWasmScreen::QWasmScreen(const emscripten::val &canvas)
     : m_canvas(canvas)
+    , m_compositor(new QWasmCompositor(this))
+    , m_eventTranslator(new QWasmEventTranslator(this))
 {
-    m_compositor = new QWasmCompositor(this);
-    m_eventTranslator = new QWasmEventTranslator(this);
     updateQScreenAndCanvasRenderSize();
     m_canvas.call<void>("focus");
 }
@@ -84,12 +84,12 @@ QWasmScreen *QWasmScreen::get(QScreen *screen)
 
 QWasmCompositor *QWasmScreen::compositor()
 {
-    return m_compositor;
+    return m_compositor.get();
 }
 
 QWasmEventTranslator *QWasmScreen::eventTranslator()
 {
-    return m_eventTranslator;
+    return m_eventTranslator.get();
 }
 
 emscripten::val QWasmScreen::canvas() const