mirror of
https://github.com/aphrody-labs/cef.git
synced 2026-10-14 05:09:23 +00:00
76 lines
3.3 KiB
Diff
76 lines
3.3 KiB
Diff
diff --git base/message_loop/message_pump_win.cc base/message_loop/message_pump_win.cc
|
|
index 5396965a64edf2d03d5624a1eaded0689715bc5a..beddc73d76b18624a44a5c27f2a7d1c67dc072ac 100644
|
|
--- base/message_loop/message_pump_win.cc
|
|
+++ base/message_loop/message_pump_win.cc
|
|
@@ -2,6 +2,7 @@
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
// found in the LICENSE file.
|
|
|
|
+#include "base/task/current_thread.h"
|
|
#include "base/message_loop/message_pump_win.h"
|
|
|
|
#include <windows.h>
|
|
@@ -611,7 +612,17 @@ bool MessagePumpForUI::ProcessNextWindowsMessage() {
|
|
if (g_pump_peek_message_with_observer && native_event_observer_) {
|
|
native_event_observer_->WillRunNativeEvent(next_peek_message_event_id_);
|
|
}
|
|
- has_msg = ::PeekMessage(&msg, nullptr, 0, 0, PM_REMOVE) != FALSE;
|
|
+
|
|
+ // We should not process all window messages if we are in the context of an
|
|
+ // OS modal loop, i.e. in the context of a windows API call like MessageBox.
|
|
+ // This is to ensure that these messages are peeked out by the OS modal loop.
|
|
+ if (CurrentThread::Get()->os_modal_loop()) {
|
|
+ // We only peek out WM_PAINT and WM_TIMER here for reasons mentioned above.
|
|
+ has_msg = PeekMessage(&msg, NULL, WM_PAINT, WM_PAINT, PM_REMOVE) ||
|
|
+ PeekMessage(&msg, NULL, WM_TIMER, WM_TIMER, PM_REMOVE);
|
|
+ } else {
|
|
+ has_msg = PeekMessage(&msg, NULL, 0, 0, PM_REMOVE) != FALSE;
|
|
+ }
|
|
if (g_pump_peek_message_with_observer && native_event_observer_) {
|
|
native_event_observer_->DidRunNativeEvent(
|
|
next_peek_message_event_id_++);
|
|
diff --git base/task/current_thread.cc base/task/current_thread.cc
|
|
index ec901a8a1dd3c73165ebb2380507b46af83ec10e..4fbd1882c6974ee7d744d263dad621af04cc3f9a 100644
|
|
--- base/task/current_thread.cc
|
|
+++ base/task/current_thread.cc
|
|
@@ -51,6 +51,8 @@ void CurrentThread::AddDestructionObserver(
|
|
|
|
void CurrentThread::RemoveDestructionObserver(
|
|
DestructionObserver* destruction_observer) {
|
|
+ if (!current_)
|
|
+ return;
|
|
DCHECK(current_->IsBoundToCurrentThread());
|
|
current_->RemoveDestructionObserver(destruction_observer);
|
|
}
|
|
diff --git base/task/current_thread.h base/task/current_thread.h
|
|
index 2bf2f919fe5cba57049cb7e80a92603c8d4980b8..72bde82f6b62e3981029d7c9ca876c853b563298 100644
|
|
--- base/task/current_thread.h
|
|
+++ base/task/current_thread.h
|
|
@@ -160,6 +160,12 @@ class BASE_EXPORT CurrentThread {
|
|
RegisterOnNextIdleCallbackPasskey,
|
|
OnceClosure on_next_idle_callback);
|
|
|
|
+#if BUILDFLAG(IS_WIN)
|
|
+ void set_os_modal_loop(bool os_modal_loop) { os_modal_loop_ = os_modal_loop; }
|
|
+
|
|
+ bool os_modal_loop() const { return os_modal_loop_; }
|
|
+#endif // OS_WIN
|
|
+
|
|
// Enables nested task processing in scope of an upcoming native message loop.
|
|
// Some unwanted message loops may occur when using common controls or printer
|
|
// functions. Hence, nested task processing is disabled by default to avoid
|
|
@@ -233,6 +239,13 @@ class BASE_EXPORT CurrentThread {
|
|
friend class web::WebTaskEnvironment;
|
|
|
|
raw_ptr<sequence_manager::internal::SequenceManagerImpl> current_;
|
|
+
|
|
+#if BUILDFLAG(IS_WIN)
|
|
+ private:
|
|
+ // Should be set to true before calling Windows APIs like TrackPopupMenu, etc.
|
|
+ // which enter a modal message loop.
|
|
+ bool os_modal_loop_ = false;
|
|
+#endif
|
|
};
|
|
|
|
// UI extension of CurrentThread.
|