diff options
| author | Alexandru Croitor <alexandru.croitor@qt.io> | 2017-11-20 10:42:27 +0100 |
|---|---|---|
| committer | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2017-12-05 17:04:18 +0100 |
| commit | 67b94abee68455b0481360ee9e9706377e6fc871 (patch) | |
| tree | 519fd7bcc3fde809208bef07b2d650d9067506e5 | |
| parent | c2d1b1b49536ffb0f3be77c21f48295c8be77346 (diff) | |
| download | qtwebengine-chromium-67b94abee68455b0481360ee9e9706377e6fc871.tar.gz | |
Use default MessageLoop type for NetworkConfigWatcherMac thread
This ensures that the NetworkChangeNotifierMac::ReachabilityCallback
is actually called from the native CoreFoundation event loop. This
was not the case before because WebEngine overrides the TYPE_UI
message loop with a QEventDispatcherUNIX, which does not integrate
a CoreFoundation loop within, thus never calling the respective
callback.
Task-number: QTBUG-61364
Change-Id: Id07bb46dfeb114e4b434acebb93c07d338bb3d1f
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
| -rw-r--r-- | chromium/net/base/network_config_watcher_mac.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chromium/net/base/network_config_watcher_mac.cc b/chromium/net/base/network_config_watcher_mac.cc index bc46c120640..ef61ad3e986 100644 --- a/chromium/net/base/network_config_watcher_mac.cc +++ b/chromium/net/base/network_config_watcher_mac.cc @@ -121,7 +121,7 @@ NetworkConfigWatcherMac::NetworkConfigWatcherMac(Delegate* delegate) // We create this notifier thread because the notification implementation // needs a thread with a CFRunLoop, and there's no guarantee that // MessageLoop::current() meets that criterion. - base::Thread::Options thread_options(base::MessageLoop::TYPE_UI, 0); + base::Thread::Options thread_options(base::MessageLoop::TYPE_DEFAULT, 0); notifier_thread_->StartWithOptions(thread_options); } |