#include "wx/msw/private/msgdlg.h"
#include "wx/progdlg.h"
+#include "wx/evtloop.h"
using namespace wxMSWMessageDialog;
LONG_PTR dwRefData);
};
-// ============================================================================
-// Helper functions
-// ============================================================================
-
namespace
{
-// This function returns true if the progress dialog with the given style
-// (combination of wxPD_XXX constants) needs the "Close" button and this button
-// only, i.e. not a "Cancel" one.
-bool UsesCloseButtonOnly(long style)
+// A custom event loop which runs until the state of the dialog becomes
+// "Dismissed".
+class wxProgressDialogModalLoop : public wxEventLoop
{
- return !(style & (wxPD_CAN_ABORT | wxPD_AUTO_HIDE));
-}
+public:
+ wxProgressDialogModalLoop(wxProgressDialogSharedData& data)
+ : m_data(data)
+ {
+ }
+
+protected:
+ virtual void OnNextIteration()
+ {
+ wxCriticalSectionLocker locker(m_data.m_cs);
+
+ if ( m_data.m_state == wxProgressDialog::Dismissed )
+ Exit();
+ }
+
+ wxProgressDialogSharedData& m_data;
+
+ wxDECLARE_NO_COPY_CLASS(wxProgressDialogModalLoop);
+};
+
+// ============================================================================
+// Helper functions
+// ============================================================================
BOOL CALLBACK DisplayCloseButton(HWND hwnd, LPARAM lParam)
{
m_title(title)
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
{
SetMaximum(maximum);
bool wxProgressDialog::Update(int value, const wxString& newmsg, bool *skip)
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
{
- wxCriticalSectionLocker locker(m_sharedData->m_cs);
+ {
+ wxCriticalSectionLocker locker(m_sharedData->m_cs);
- // Do nothing in canceled state.
- if ( !DoNativeBeforeUpdate(skip) )
- return false;
+ // Do nothing in canceled state.
+ if ( !DoNativeBeforeUpdate(skip) )
+ return false;
- value /= m_factor;
+ value /= m_factor;
- wxASSERT_MSG( value <= m_maximum, wxT("invalid progress value") );
+ wxASSERT_MSG( value <= m_maximum, wxT("invalid progress value") );
- m_sharedData->m_value = value;
- m_sharedData->m_notifications |= wxSPDD_VALUE_CHANGED;
+ m_sharedData->m_value = value;
+ m_sharedData->m_notifications |= wxSPDD_VALUE_CHANGED;
- if ( !newmsg.empty() )
- {
- m_message = newmsg;
- m_sharedData->m_message = newmsg;
- m_sharedData->m_notifications |= wxSPDD_MESSAGE_CHANGED;
- }
+ if ( !newmsg.empty() )
+ {
+ m_message = newmsg;
+ m_sharedData->m_message = newmsg;
+ m_sharedData->m_notifications |= wxSPDD_MESSAGE_CHANGED;
+ }
- if ( m_sharedData->m_progressBarMarquee )
- {
- m_sharedData->m_progressBarMarquee = false;
- m_sharedData->m_notifications |= wxSPDD_PBMARQUEE_CHANGED;
- }
+ if ( m_sharedData->m_progressBarMarquee )
+ {
+ m_sharedData->m_progressBarMarquee = false;
+ m_sharedData->m_notifications |= wxSPDD_PBMARQUEE_CHANGED;
+ }
- UpdateExpandedInformation( value );
+ UpdateExpandedInformation( value );
- // Has the progress bar finished?
- if ( value == m_maximum )
- {
- if ( m_state == Finished )
- return true;
+ // If we didn't just reach the finish, all we have to do is to
+ // return true if the dialog wasn't cancelled and false otherwise.
+ if ( value != m_maximum || m_state == Finished )
+ return m_sharedData->m_state != Canceled;
+
+ // On finishing, the dialog without wxPD_AUTO_HIDE style becomes a
+ // modal one meaning that we must block here until the user
+ // dismisses it.
m_state = Finished;
m_sharedData->m_state = Finished;
m_sharedData->m_notifications |= wxSPDD_FINISHED;
- if( !HasPDFlag(wxPD_AUTO_HIDE) && newmsg.empty() )
+ if ( HasPDFlag(wxPD_AUTO_HIDE) )
+ return true;
+
+ if ( newmsg.empty() )
{
// Provide the finishing message if the application didn't.
m_message = _("Done.");
m_sharedData->m_message = m_message;
m_sharedData->m_notifications |= wxSPDD_MESSAGE_CHANGED;
}
- }
-
- return m_sharedData->m_state != Canceled;
+ } // unlock m_sharedData->m_cs
+
+ // We only get here when we need to wait for the dialog to terminate so
+ // do just this by running a custom event loop until the dialog is
+ // dismissed.
+ wxProgressDialogModalLoop loop(*m_sharedData);
+ loop.Run();
+ return true;
}
#endif // wxHAS_MSW_TASKDIALOG
bool wxProgressDialog::Pulse(const wxString& newmsg, bool *skip)
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
{
wxCriticalSectionLocker locker(m_sharedData->m_cs);
bool wxProgressDialog::DoNativeBeforeUpdate(bool *skip)
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
{
if ( m_sharedData->m_skipped )
{
wxGenericProgressDialog::Resume();
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
{
HWND hwnd;
// it now.
m_sharedData->m_notifications |= wxSPDD_ENABLE_SKIP;
- if ( !UsesCloseButtonOnly(GetPDStyle()) )
+ // Also re-enable "Cancel" itself
+ if ( HasPDFlag(wxPD_CAN_ABORT) )
m_sharedData->m_notifications |= wxSPDD_ENABLE_ABORT;
hwnd = m_sharedData->m_hwnd;
int wxProgressDialog::GetValue() const
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
{
wxCriticalSectionLocker locker(m_sharedData->m_cs);
return m_sharedData->m_value;
wxString wxProgressDialog::GetMessage() const
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
return m_message;
#endif // wxHAS_MSW_TASKDIALOG
void wxProgressDialog::SetRange(int maximum)
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
{
SetMaximum(maximum);
bool wxProgressDialog::WasSkipped() const
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
{
if ( !m_sharedData )
{
bool wxProgressDialog::WasCancelled() const
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
{
wxCriticalSectionLocker locker(m_sharedData->m_cs);
return m_sharedData->m_state == Canceled;
void wxProgressDialog::SetTitle(const wxString& title)
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
{
m_title = title;
wxString wxProgressDialog::GetTitle() const
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
return m_title;
#endif // wxHAS_MSW_TASKDIALOG
bool wxProgressDialog::Show(bool show)
{
#ifdef wxHAS_MSW_TASKDIALOG
- if ( HasNativeProgressDialog() )
+ if ( HasNativeTaskDialog() )
{
// The dialog can't be hidden at all and showing it again after it had
// been shown before doesn't do anything.
m_sharedData->m_state = Continue;
m_sharedData->m_labelCancel = _("Cancel");
}
- else if ( !HasPDFlag(wxPD_AUTO_HIDE) )
+ else // Dialog can't be cancelled.
{
+ // We still must have at least a single button in the dialog so
+ // just don't call it "Cancel" in this case.
m_sharedData->m_labelCancel = _("Close");
}
return wxGenericProgressDialog::Show( show );
}
-bool wxProgressDialog::HasNativeProgressDialog() const
-{
-#ifdef wxHAS_MSW_TASKDIALOG
- // Native task dialog, if available, can't be used without any buttons so
- // we fall back to the generic one if none of "Skip", "Cancel" and "Close"
- // buttons is used.
- return HasNativeTaskDialog()
- && (HasPDFlag(wxPD_CAN_SKIP | wxPD_CAN_ABORT) ||
- !HasPDFlag(wxPD_AUTO_HIDE));
-#else // !wxHAS_MSW_TASKDIALOG
- // This shouldn't be even called in !wxHAS_MSW_TASKDIALOG case but as we
- // still must define the function as returning something, return false.
- return false;
-#endif // wxHAS_MSW_TASKDIALOG/!wxHAS_MSW_TASKDIALOG
-}
-
void wxProgressDialog::UpdateExpandedInformation(int value)
{
#ifdef wxHAS_MSW_TASKDIALOG
// Use a Cancel button when requested or use a Close button when
// the dialog does not automatically hide.
- if ( (m_sharedData.m_style & wxPD_CAN_ABORT)
- || !(m_sharedData.m_style & wxPD_AUTO_HIDE) )
- {
- wxTdc.AddTaskDialogButton( tdc, IDCANCEL, 0,
- m_sharedData.m_labelCancel );
- }
+ wxTdc.AddTaskDialogButton( tdc, IDCANCEL, 0,
+ m_sharedData.m_labelCancel );
tdc.dwFlags |= TDF_CALLBACK_TIMER | TDF_SHOW_PROGRESS_BAR;
if ( FAILED(hr) )
wxLogApiError( "TaskDialogIndirect", hr );
+ // If the main thread is waiting for us to exit inside the event loop in
+ // Update(), wake it up so that it checks our status again.
+ wxWakeUpIdle();
+
return NULL;
}
0,
MAKELPARAM(0, sharedData->m_range) );
- if ( UsesCloseButtonOnly(sharedData->m_style) )
+ // If we can't be aborted, the "Close" button will only be enabled
+ // when the progress ends (and not even then with wxPD_AUTO_HIDE).
+ if ( !(sharedData->m_style & wxPD_CAN_ABORT) )
::SendMessage( hwnd, TDM_ENABLE_BUTTON, IDCANCEL, FALSE );
break;
case IDCANCEL:
if ( sharedData->m_state == wxProgressDialog::Finished )
{
+ // If the main thread is waiting for us, tell it that
+ // we're gone (and if it doesn't wait, it's harmless).
+ sharedData->m_state = wxProgressDialog::Dismissed;
+
+ // Let Windows close the dialog.
return FALSE;
}
// Close button on the window triggers an IDCANCEL press,
// don't allow it when it should only be possible to close
// a finished dialog.
- if ( !UsesCloseButtonOnly(sharedData->m_style) )
+ if ( sharedData->m_style & wxPD_CAN_ABORT )
{
wxCHECK_MSG
(
case TDN_TIMER:
PerformNotificationUpdates(hwnd, sharedData);
- // End dialog in three different cases:
- // 1. Progress finished and dialog should automatically hide.
- // 2. The wxProgressDialog object was destructed and should
- // automatically hide.
- // 3. The dialog was canceled and wxProgressDialog object
- // was destroyed.
- bool isCanceled =
- sharedData->m_state == wxGenericProgressDialog::Canceled;
- bool isFinished =
- sharedData->m_state == wxGenericProgressDialog::Finished;
- bool wasDestroyed =
- (sharedData->m_notifications & wxSPDD_DESTROYED) != 0;
- bool shouldAutoHide = (sharedData->m_style & wxPD_AUTO_HIDE) != 0;
-
- if ( (shouldAutoHide && (isFinished || wasDestroyed))
- || (wasDestroyed && isCanceled) )
+ /*
+ Decide whether we should end the dialog. This is done if either
+ the dialog object itself was destroyed or if the progress
+ finished and we were configured to hide automatically without
+ waiting for the user to dismiss us.
+
+ Notice that we do not close the dialog if it was cancelled
+ because it's up to the user code in the main thread to decide
+ whether it really wants to cancel the dialog.
+ */
+ if ( (sharedData->m_notifications & wxSPDD_DESTROYED) ||
+ (sharedData->m_state == wxProgressDialog::Finished &&
+ sharedData->m_style & wxPD_AUTO_HIDE) )
{
::EndDialog( hwnd, IDCLOSE );
}
projects / wxWidgets.git / blobdiff