AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

WinRT: Fix native MessageBox not closing

Browse Source 
The message box buttons need to be added inside the XAML thread.
Otherwise QWinRTMessageDialogHelper::onCompleted will not be triggered.

To successfully emit the clicked signal across different threads do not
exit the eventloop.

Change-Id: Ie884bdfe0dc64132559755083dae50c2aa43d80b
Task-number: QTBUG-48209
Reviewed-by: Oliver Wolff <oliver.wolff@theqtcompany.com>
This commit is contained in:
Maurice Kalinowski 2015-09-21 20:40:13 +02:00
parent 676edc006e
commit 72d0c62d14
1 changed files with 40 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/plugins/platforms/winrt/qwinrtmessagedialoghelper.cpp
View File

@ -133,10 +133,13 @@ bool QWinRTMessageDialogHelper::show(Qt::WindowFlags windowFlags, Qt::WindowModa
RETURN_FALSE_IF_FAILED("Failed to create dialog"); RETURN_FALSE_IF_FAILED("Failed to create dialog");
} }
hr = QEventDispatcherWinRT::runOnXamlThread([this, d, options, commandFactory, dialog]() {
HRESULT hr;
// Add Buttons // Add Buttons
ComPtr<IVector<IUICommand *>> dialogCommands; ComPtr<IVector<IUICommand *>> dialogCommands;
hr = dialog->get_Commands(&dialogCommands); hr = dialog->get_Commands(&dialogCommands);
RETURN_FALSE_IF_FAILED("Failed to get dialog commands"); RETURN_HR_IF_FAILED("Failed to get dialog commands");
// If no button is specified we need to create one to get close notification // If no button is specified we need to create one to get close notification
int buttons = options->standardButtons(); int buttons = options->standardButtons();
@ -151,27 +154,25 @@ bool QWinRTMessageDialogHelper::show(Qt::WindowFlags windowFlags, Qt::WindowModa
HStringReference nativeLabel(reinterpret_cast<LPCWSTR>(label.utf16()), label.size()); HStringReference nativeLabel(reinterpret_cast<LPCWSTR>(label.utf16()), label.size());
ComPtr<IUICommand> command; ComPtr<IUICommand> command;
hr = commandFactory->Create(nativeLabel.Get(), &command); hr = commandFactory->Create(nativeLabel.Get(), &command);
RETURN_FALSE_IF_FAILED("Failed to create message box command"); RETURN_HR_IF_FAILED("Failed to create message box command");
ComPtr<IInspectable> id = Make<CommandId>(static_cast<StandardButton>(i)); ComPtr<IInspectable> id = Make<CommandId>(static_cast<StandardButton>(i));
hr = command->put_Id(id.Get()); hr = command->put_Id(id.Get());
RETURN_FALSE_IF_FAILED("Failed to set command ID"); RETURN_HR_IF_FAILED("Failed to set command ID");
hr = dialogCommands->Append(command.Get()); hr = dialogCommands->Append(command.Get());
if (hr == E_BOUNDS) { if (hr == E_BOUNDS) {
qErrnoWarning(hr, "The WinRT message dialog supports a maximum of three buttons"); qErrnoWarning(hr, "The WinRT message dialog supports a maximum of three buttons");
continue; continue;
} }
RETURN_FALSE_IF_FAILED("Failed to append message box command"); RETURN_HR_IF_FAILED("Failed to append message box command");
if (i == Abort || i == Cancel || i == Close) { if (i == Abort || i == Cancel || i == Close) {
quint32 size; quint32 size;
hr = dialogCommands->get_Size(&size); hr = dialogCommands->get_Size(&size);
RETURN_FALSE_IF_FAILED("Failed to get command list size"); RETURN_HR_IF_FAILED("Failed to get command list size");
hr = dialog->put_CancelCommandIndex(size - 1); hr = dialog->put_CancelCommandIndex(size - 1);
RETURN_FALSE_IF_FAILED("Failed to set cancel index"); RETURN_HR_IF_FAILED("Failed to set cancel index");
} }
} }
hr = QEventDispatcherWinRT::runOnXamlThread([this, d, dialog]() {
HRESULT hr;
ComPtr<IAsyncOperation<IUICommand *>> op; ComPtr<IAsyncOperation<IUICommand *>> op;
hr = dialog->ShowAsync(&op); hr = dialog->ShowAsync(&op);
RETURN_HR_IF_FAILED("Failed to show dialog"); RETURN_HR_IF_FAILED("Failed to show dialog");
@ -206,8 +207,7 @@ HRESULT QWinRTMessageDialogHelper::onCompleted(IAsyncOperation<IUICommand *> *as
Q_UNUSED(status); Q_UNUSED(status);
Q_D(QWinRTMessageDialogHelper); Q_D(QWinRTMessageDialogHelper);
if (d->loop.isRunning()) QEventLoopLocker locker(&d->loop);
d->loop.exit();
d->shown = false; d->shown = false;