winrt: Clean up platform message dialog helper

- Remove "Platform" from class name for consistency
- Use ComPtr and HRESULT checking where missing
- Use UICommand ID class to reduce callback complexity
- Use dialog completed callback to fix failed repeated dialog opening

These changes have been tested with the QtQuick.Dialogs message dialog
example, and all features appear to be working. Note that the WinRT
dialog supports a maximum of three buttons, though, and a warning is
printed if this number is exceeded.

Similarly to Android, the native hooks can be now be disabled by
using qputenv("QT_USE_WINRT_NATIVE_DIALOGS", "0").

Task-number: QTBUG-38115
Task-number: QTBUG-39868
Change-Id: I9943c7c11bd640790db68219cefdca1a680e96ec
Reviewed-by: Maurice Kalinowski <maurice.kalinowski@digia.com>

1 files changed, 4 insertions, 2 deletions

diff --git a/src/plugins/platforms/winrt/winrt.pro b/src/plugins/platforms/winrt/winrt.pro
index 5460a7b18c..427920a670 100644
--- a/src/plugins/platforms/winrt/winrt.pro
+++ b/src/plugins/platforms/winrt/winrt.pro
@@ -35,12 +35,13 @@ SOURCES = \
     qwinrtfontdatabase.cpp \
     qwinrtinputcontext.cpp \
     qwinrtintegration.cpp \
-    qwinrtplatformmessagedialoghelper.cpp \
+    qwinrtmessagedialoghelper.cpp \
     qwinrtscreen.cpp \
     qwinrtservices.cpp \
     qwinrttheme.cpp \
     qwinrtwindow.cpp
 
+
 HEADERS = \
     qwinrtbackingstore.h \
     qwinrtcursor.h \
@@ -49,12 +50,13 @@ HEADERS = \
     qwinrtfontdatabase.h \
     qwinrtinputcontext.h \
     qwinrtintegration.h \
-    qwinrtplatformmessagedialoghelper.h \
+    qwinrtmessagedialoghelper.h \
     qwinrtscreen.h \
     qwinrtservices.h \
     qwinrttheme.h \
     qwinrtwindow.h
 
+
 BLIT_INPUT = $$PWD/blit.hlsl
 fxc_blitps.commands = fxc.exe /nologo /T ps_4_0_level_9_1 /E blitps /Vn q_blitps /Fh ${QMAKE_FILE_OUT} ${QMAKE_FILE_NAME}
 fxc_blitps.output = $$OUT_PWD/blitps.h