|
|
8d06c8b |
|
|
|
8d06c8b |
# HG changeset patch
|
|
|
8d06c8b |
# User sotaro <sotaro.ikeda.g@gmail.com>
|
|
|
8d06c8b |
# Date 1610634595 0
|
|
|
8d06c8b |
# Node ID c989e16ae8d0801b76efe712658abcbf3704a486
|
|
|
8d06c8b |
# Parent dc0d1d98e111aa781333980c2561f534ea1ebb0b
|
|
|
8d06c8b |
Bug 1681107 - Fix race condition of calling CompositorBridgeChild::SendPause() r=rmader,stransky
|
|
|
8d06c8b |
|
|
|
8d06c8b |
Differential Revision: https://phabricator.services.mozilla.com/D101693
|
|
|
8d06c8b |
|
|
|
8d06c8b |
diff --git a/widget/gtk/nsWindow.cpp b/widget/gtk/nsWindow.cpp
|
|
|
8d06c8b |
--- a/widget/gtk/nsWindow.cpp
|
|
|
8d06c8b |
+++ b/widget/gtk/nsWindow.cpp
|
|
|
8d06c8b |
@@ -5085,23 +5085,29 @@ void nsWindow::NativeMoveResize() {
|
|
|
8d06c8b |
if (mNeedsShow && mIsShown) {
|
|
|
8d06c8b |
NativeShow(true);
|
|
|
8d06c8b |
}
|
|
|
8d06c8b |
}
|
|
|
8d06c8b |
|
|
|
8d06c8b |
void nsWindow::PauseRemoteRenderer() {
|
|
|
8d06c8b |
#ifdef MOZ_WAYLAND
|
|
|
8d06c8b |
if (!mIsDestroyed) {
|
|
|
8d06c8b |
- if (mContainer && moz_container_wayland_has_egl_window(mContainer)) {
|
|
|
8d06c8b |
+ if (mContainer) {
|
|
|
8d06c8b |
// Because wl_egl_window is destroyed on moz_container_unmap(),
|
|
|
8d06c8b |
// the current compositor cannot use it anymore. To avoid crash,
|
|
|
8d06c8b |
// pause the compositor and destroy EGLSurface & resume the compositor
|
|
|
8d06c8b |
// and re-create EGLSurface on next expose event.
|
|
|
8d06c8b |
- MOZ_ASSERT(GetRemoteRenderer());
|
|
|
8d06c8b |
- if (CompositorBridgeChild* remoteRenderer = GetRemoteRenderer()) {
|
|
|
8d06c8b |
+
|
|
|
8d06c8b |
+ // moz_container_wayland_has_egl_window() could not be used here, since
|
|
|
8d06c8b |
+ // there is a case that resume compositor is not completed yet.
|
|
|
8d06c8b |
+
|
|
|
8d06c8b |
+ CompositorBridgeChild* remoteRenderer = GetRemoteRenderer();
|
|
|
8d06c8b |
+ bool needsCompositorPause = !mNeedsCompositorResume && !!remoteRenderer &&
|
|
|
8d06c8b |
+ mCompositorWidgetDelegate;
|
|
|
8d06c8b |
+ if (needsCompositorPause) {
|
|
|
8d06c8b |
// XXX slow sync IPC
|
|
|
8d06c8b |
remoteRenderer->SendPause();
|
|
|
8d06c8b |
// Re-request initial draw callback
|
|
|
8d06c8b |
RefPtr<nsWindow> self(this);
|
|
|
8d06c8b |
moz_container_wayland_add_initial_draw_callback(
|
|
|
8d06c8b |
mContainer, [self]() -> void {
|
|
|
8d06c8b |
self->mNeedsCompositorResume = true;
|
|
|
8d06c8b |
self->MaybeResumeCompositor();
|
|
|
8d06c8b |
|