[mutter] Explicitly create per backend stage windows
- From: Jonas Ã…dahl <jadahl src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [mutter] Explicitly create per backend stage windows
- Date: Wed, 20 Jul 2016 06:27:38 +0000 (UTC)
commit c3d2352a9e848ee9797406014ce4f494cddd19e6
Author: Jonas Ã…dahl <jadahl gmail com>
Date: Fri May 20 15:58:10 2016 +0800
Explicitly create per backend stage windows
Split the stage window implementations into three separate objects: one
for X11 as a compositing manager, one for X11 running as a nested
Wayland compositor, and one for running with the native backend.
The new stage window implementations are only thin shells; this is in
preparation for making the stage windows behave more differently.
https://bugzilla.gnome.org/show_bug.cgi?id=768976
clutter/clutter/clutter-backend-private.h | 5 +-
clutter/clutter/clutter-backend.c | 34 ----------------
clutter/clutter/clutter-event-translator.h | 1 +
clutter/clutter/clutter-mutter.h | 2 +
clutter/clutter/egl/clutter-backend-eglnative.c | 3 -
clutter/clutter/egl/clutter-stage-eglnative.h | 3 +
clutter/clutter/x11/clutter-backend-x11.c | 3 +-
clutter/clutter/x11/clutter-stage-x11.h | 3 +
src/Makefile.am | 4 ++
src/backends/native/meta-clutter-backend-native.c | 13 ++++++
src/backends/native/meta-stage-native.c | 45 +++++++++++++++++++++
src/backends/native/meta-stage-native.h | 34 ++++++++++++++++
src/backends/x11/meta-clutter-backend-x11.c | 29 +++++++++++++
src/backends/x11/meta-stage-x11-nested.c | 45 +++++++++++++++++++++
src/backends/x11/meta-stage-x11-nested.h | 34 ++++++++++++++++
15 files changed, 217 insertions(+), 41 deletions(-)
---
diff --git a/clutter/clutter/clutter-backend-private.h b/clutter/clutter/clutter-backend-private.h
index a2b729c..f485cba 100644
--- a/clutter/clutter/clutter-backend-private.h
+++ b/clutter/clutter/clutter-backend-private.h
@@ -65,8 +65,6 @@ struct _ClutterBackendClass
/*< private >*/
GObjectClass parent_class;
- GType stage_window_type;
-
/* vfuncs */
gboolean (* pre_parse) (ClutterBackend *backend,
GError **error);
@@ -138,8 +136,11 @@ void _clutter_backend_free_event_data (Clutter
gboolean _clutter_backend_translate_event (ClutterBackend *backend,
gpointer native,
ClutterEvent *event);
+
+CLUTTER_AVAILABLE_IN_MUTTER
void _clutter_backend_add_event_translator (ClutterBackend *backend,
ClutterEventTranslator *translator);
+
void _clutter_backend_remove_event_translator (ClutterBackend *backend,
ClutterEventTranslator *translator);
diff --git a/clutter/clutter/clutter-backend.c b/clutter/clutter/clutter-backend.c
index 0f38270..c3b7295 100644
--- a/clutter/clutter/clutter-backend.c
+++ b/clutter/clutter/clutter-backend.c
@@ -467,37 +467,6 @@ clutter_backend_real_get_features (ClutterBackend *backend)
return flags;
}
-static ClutterStageWindow *
-clutter_backend_real_create_stage (ClutterBackend *backend,
- ClutterStage *wrapper,
- GError **error)
-{
- ClutterBackendClass *klass;
-
- if (!clutter_feature_available (CLUTTER_FEATURE_STAGE_MULTIPLE))
- {
- ClutterStageManager *manager = clutter_stage_manager_get_default ();
-
- if (clutter_stage_manager_get_default_stage (manager) != NULL)
- {
- g_set_error (error, CLUTTER_INIT_ERROR,
- CLUTTER_INIT_ERROR_BACKEND,
- _("The backend of type '%s' does not support "
- "creating multiple stages"),
- G_OBJECT_TYPE_NAME (backend));
- return NULL;
- }
- }
-
- klass = CLUTTER_BACKEND_GET_CLASS (backend);
- g_assert (klass->stage_window_type != G_TYPE_INVALID);
-
- return g_object_new (klass->stage_window_type,
- "backend", backend,
- "wrapper", wrapper,
- NULL);
-}
-
static const char *allowed_backends;
static ClutterBackend * (* custom_backend_func) (void);
@@ -667,8 +636,6 @@ clutter_backend_class_init (ClutterBackendClass *klass)
gobject_class->dispose = clutter_backend_dispose;
gobject_class->finalize = clutter_backend_finalize;
- klass->stage_window_type = G_TYPE_INVALID;
-
/**
* ClutterBackend::resolution-changed:
* @backend: the #ClutterBackend that emitted the signal
@@ -732,7 +699,6 @@ clutter_backend_class_init (ClutterBackendClass *klass)
klass->create_context = clutter_backend_real_create_context;
klass->ensure_context = clutter_backend_real_ensure_context;
klass->get_features = clutter_backend_real_get_features;
- klass->create_stage = clutter_backend_real_create_stage;
}
static void
diff --git a/clutter/clutter/clutter-event-translator.h b/clutter/clutter/clutter-event-translator.h
index f7facc7..e784fd9 100644
--- a/clutter/clutter/clutter-event-translator.h
+++ b/clutter/clutter/clutter-event-translator.h
@@ -29,6 +29,7 @@ struct _ClutterEventTranslatorIface
ClutterEvent *translated);
};
+CLUTTER_AVAILABLE_IN_MUTTER
GType _clutter_event_translator_get_type (void) G_GNUC_CONST;
ClutterTranslateReturn _clutter_event_translator_translate_event (ClutterEventTranslator *translator,
diff --git a/clutter/clutter/clutter-mutter.h b/clutter/clutter/clutter-mutter.h
index ebd06b6..fec6214 100644
--- a/clutter/clutter/clutter-mutter.h
+++ b/clutter/clutter/clutter-mutter.h
@@ -27,6 +27,8 @@
#include "clutter-backend.h"
#include "clutter-macros.h"
+#include "egl/clutter-stage-eglnative.h"
+#include "x11/clutter-stage-x11.h"
CLUTTER_AVAILABLE_IN_MUTTER
void clutter_set_custom_backend_func (ClutterBackend *(* func) (void));
diff --git a/clutter/clutter/egl/clutter-backend-eglnative.c b/clutter/clutter/egl/clutter-backend-eglnative.c
index 649110b..cbe2067 100644
--- a/clutter/clutter/egl/clutter-backend-eglnative.c
+++ b/clutter/clutter/egl/clutter-backend-eglnative.c
@@ -75,11 +75,8 @@ static void
clutter_backend_egl_native_class_init (ClutterBackendEglNativeClass *klass)
{
GObjectClass *gobject_class = G_OBJECT_CLASS (klass);
- ClutterBackendClass *backend_class = CLUTTER_BACKEND_CLASS (klass);
gobject_class->dispose = clutter_backend_egl_native_dispose;
-
- backend_class->stage_window_type = CLUTTER_TYPE_STAGE_EGL_NATIVE;
}
static void
diff --git a/clutter/clutter/egl/clutter-stage-eglnative.h b/clutter/clutter/egl/clutter-stage-eglnative.h
index a237ffa..073ce92 100644
--- a/clutter/clutter/egl/clutter-stage-eglnative.h
+++ b/clutter/clutter/egl/clutter-stage-eglnative.h
@@ -40,6 +40,8 @@
typedef struct _ClutterStageEglNative ClutterStageEglNative;
typedef struct _ClutterStageEglNativeClass ClutterStageEglNativeClass;
+G_DEFINE_AUTOPTR_CLEANUP_FUNC (ClutterStageEglNative, g_object_unref)
+
struct _ClutterStageEglNative
{
ClutterStageCogl parent_instance;
@@ -50,6 +52,7 @@ struct _ClutterStageEglNativeClass
ClutterStageCoglClass parent_class;
};
+CLUTTER_AVAILABLE_IN_MUTTER
GType _clutter_stage_eglnative_get_type (void) G_GNUC_CONST;
#endif /* __CLUTTER_STAGE_EGL_NATIVE_H__ */
diff --git a/clutter/clutter/x11/clutter-backend-x11.c b/clutter/clutter/x11/clutter-backend-x11.c
index 4a299cf..50fe293 100644
--- a/clutter/clutter/x11/clutter-backend-x11.c
+++ b/clutter/clutter/x11/clutter-backend-x11.c
@@ -829,7 +829,7 @@ clutter_backend_x11_class_init (ClutterBackendX11Class *klass)
gobject_class->dispose = clutter_backend_x11_dispose;
gobject_class->finalize = clutter_backend_x11_finalize;
- backend_class->stage_window_type = CLUTTER_TYPE_STAGE_X11;
+ backend_class->create_stage = clutter_backend_x11_create_stage;
backend_class->pre_parse = clutter_backend_x11_pre_parse;
backend_class->post_parse = clutter_backend_x11_post_parse;
@@ -840,7 +840,6 @@ clutter_backend_x11_class_init (ClutterBackendX11Class *klass)
backend_class->get_renderer = clutter_backend_x11_get_renderer;
backend_class->get_display = clutter_backend_x11_get_display;
- backend_class->create_stage = clutter_backend_x11_create_stage;
backend_class->get_keymap_direction = clutter_backend_x11_get_keymap_direction;
}
diff --git a/clutter/clutter/x11/clutter-stage-x11.h b/clutter/clutter/x11/clutter-stage-x11.h
index 1e03510..5f577d8 100644
--- a/clutter/clutter/x11/clutter-stage-x11.h
+++ b/clutter/clutter/x11/clutter-stage-x11.h
@@ -42,6 +42,8 @@ G_BEGIN_DECLS
typedef struct _ClutterStageX11 ClutterStageX11;
typedef struct _ClutterStageX11Class ClutterStageX11Class;
+G_DEFINE_AUTOPTR_CLEANUP_FUNC (ClutterStageX11, g_object_unref)
+
typedef enum
{
STAGE_X11_WITHDRAWN = 1 << 1
@@ -76,6 +78,7 @@ struct _ClutterStageX11Class
ClutterStageCoglClass parent_class;
};
+CLUTTER_AVAILABLE_IN_MUTTER
GType _clutter_stage_x11_get_type (void) G_GNUC_CONST;
void _clutter_stage_x11_events_device_changed (ClutterStageX11 *stage_x11,
diff --git a/src/Makefile.am b/src/Makefile.am
index a4c7cd0..b378f9f 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -127,6 +127,8 @@ libmutter_la_SOURCES = \
backends/x11/meta-monitor-manager-xrandr.h \
backends/x11/meta-renderer-x11.c \
backends/x11/meta-renderer-x11.h \
+ backends/x11/meta-stage-x11-nested.c \
+ backends/x11/meta-stage-x11-nested.h \
core/meta-accel-parse.c \
core/meta-accel-parse.h \
meta/barrier.h \
@@ -351,6 +353,8 @@ libmutter_la_SOURCES += \
backends/native/meta-launcher.h \
backends/native/meta-renderer-native.c \
backends/native/meta-renderer-native.h \
+ backends/native/meta-stage-native.c \
+ backends/native/meta-stage-native.h \
backends/native/dbus-utils.c \
backends/native/dbus-utils.h \
$(NULL)
diff --git a/src/backends/native/meta-clutter-backend-native.c
b/src/backends/native/meta-clutter-backend-native.c
index 4371241..675395a 100644
--- a/src/backends/native/meta-clutter-backend-native.c
+++ b/src/backends/native/meta-clutter-backend-native.c
@@ -29,6 +29,7 @@
#include "backends/meta-backend-private.h"
#include "backends/meta-renderer.h"
#include "backends/native/meta-clutter-backend-native.h"
+#include "backends/native/meta-stage-native.h"
#include "clutter/clutter.h"
#include "meta/meta-backend.h"
@@ -50,6 +51,17 @@ meta_clutter_backend_native_get_renderer (ClutterBackend *clutter_backend,
return meta_renderer_create_cogl_renderer (renderer);
}
+static ClutterStageWindow *
+meta_clutter_backend_native_create_stage (ClutterBackend *backend,
+ ClutterStage *wrapper,
+ GError **error)
+{
+ return g_object_new (META_TYPE_STAGE_NATIVE,
+ "backend", backend,
+ "wrapper", wrapper,
+ NULL);
+}
+
static void
meta_clutter_backend_native_init (MetaClutterBackendNative *clutter_backend_nativen)
{
@@ -61,4 +73,5 @@ meta_clutter_backend_native_class_init (MetaClutterBackendNativeClass *klass)
ClutterBackendClass *clutter_backend_class = CLUTTER_BACKEND_CLASS (klass);
clutter_backend_class->get_renderer = meta_clutter_backend_native_get_renderer;
+ clutter_backend_class->create_stage = meta_clutter_backend_native_create_stage;
}
diff --git a/src/backends/native/meta-stage-native.c b/src/backends/native/meta-stage-native.c
new file mode 100644
index 0000000..6547bb4
--- /dev/null
+++ b/src/backends/native/meta-stage-native.c
@@ -0,0 +1,45 @@
+/* -*- mode: C; c-file-style: "gnu"; indent-tabs-mode: nil; -*- */
+
+/*
+ * Copyright (C) 2016 Red Hat Inc.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
+ * 02111-1307, USA.
+ *
+ * Written by:
+ * Jonas Ã…dahl <jadahl gmail com>
+ */
+
+#include "config.h"
+
+#include "backends/native/meta-stage-native.h"
+
+struct _MetaStageNative
+{
+ ClutterStageEglNative parent;
+};
+
+G_DEFINE_TYPE (MetaStageNative, meta_stage_native,
+ CLUTTER_TYPE_STAGE_EGL_NATIVE)
+
+static void
+meta_stage_native_init (MetaStageNative *stage_native)
+{
+}
+
+static void
+meta_stage_native_class_init (MetaStageNativeClass *klass)
+{
+}
diff --git a/src/backends/native/meta-stage-native.h b/src/backends/native/meta-stage-native.h
new file mode 100644
index 0000000..7668bbf
--- /dev/null
+++ b/src/backends/native/meta-stage-native.h
@@ -0,0 +1,34 @@
+/* -*- mode: C; c-file-style: "gnu"; indent-tabs-mode: nil; -*- */
+
+/*
+ * Copyright (C) 2016 Red Hat Inc.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
+ * 02111-1307, USA.
+ *
+ * Written by:
+ * Jonas Ã…dahl <jadahl gmail com>
+ */
+
+#ifndef META_STAGE_NATIVE_H
+#define META_STAGE_NATIVE_H
+
+#include "clutter/clutter-mutter.h"
+
+#define META_TYPE_STAGE_NATIVE (meta_stage_native_get_type ())
+G_DECLARE_FINAL_TYPE (MetaStageNative, meta_stage_native,
+ META, STAGE_NATIVE, ClutterStageEglNative)
+
+#endif /* META_STAGE_NATIVE_H */
diff --git a/src/backends/x11/meta-clutter-backend-x11.c b/src/backends/x11/meta-clutter-backend-x11.c
index a3b6a50..2e8474d 100644
--- a/src/backends/x11/meta-clutter-backend-x11.c
+++ b/src/backends/x11/meta-clutter-backend-x11.c
@@ -29,7 +29,9 @@
#include "backends/meta-backend-private.h"
#include "backends/meta-renderer.h"
#include "backends/x11/meta-clutter-backend-x11.h"
+#include "backends/x11/meta-stage-x11-nested.h"
#include "clutter/clutter.h"
+#include "clutter/clutter-mutter.h"
#include "meta/meta-backend.h"
struct _MetaClutterBackendX11
@@ -50,6 +52,32 @@ meta_clutter_backend_x11_get_renderer (ClutterBackend *clutter_backend,
return meta_renderer_create_cogl_renderer (renderer);
}
+static ClutterStageWindow *
+meta_clutter_backend_x11_create_stage (ClutterBackend *backend,
+ ClutterStage *wrapper,
+ GError **error)
+{
+ ClutterEventTranslator *translator;
+ ClutterStageWindow *stage;
+ GType stage_type;
+
+ if (meta_is_wayland_compositor ())
+ stage_type = META_TYPE_STAGE_X11_NESTED;
+ else
+ stage_type = CLUTTER_TYPE_STAGE_X11;
+
+ stage = g_object_new (stage_type,
+ "backend", backend,
+ "wrapper", wrapper,
+ NULL);
+
+ /* the X11 stage does event translation */
+ translator = CLUTTER_EVENT_TRANSLATOR (stage);
+ _clutter_backend_add_event_translator (backend, translator);
+
+ return stage;
+}
+
static void
meta_clutter_backend_x11_init (MetaClutterBackendX11 *clutter_backend_x11)
{
@@ -61,4 +89,5 @@ meta_clutter_backend_x11_class_init (MetaClutterBackendX11Class *klass)
ClutterBackendClass *clutter_backend_class = CLUTTER_BACKEND_CLASS (klass);
clutter_backend_class->get_renderer = meta_clutter_backend_x11_get_renderer;
+ clutter_backend_class->create_stage = meta_clutter_backend_x11_create_stage;
}
diff --git a/src/backends/x11/meta-stage-x11-nested.c b/src/backends/x11/meta-stage-x11-nested.c
new file mode 100644
index 0000000..ba0173f
--- /dev/null
+++ b/src/backends/x11/meta-stage-x11-nested.c
@@ -0,0 +1,45 @@
+/* -*- mode: C; c-file-style: "gnu"; indent-tabs-mode: nil; -*- */
+
+/*
+ * Copyright (C) 2016 Red Hat Inc.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
+ * 02111-1307, USA.
+ *
+ * Written by:
+ * Jonas Ã…dahl <jadahl gmail com>
+ */
+
+#include "config.h"
+
+#include "backends/x11/meta-stage-x11-nested.h"
+
+struct _MetaStageX11Nested
+{
+ ClutterStageX11 parent;
+};
+
+G_DEFINE_TYPE (MetaStageX11Nested, meta_stage_x11_nested,
+ CLUTTER_TYPE_STAGE_X11)
+
+static void
+meta_stage_x11_nested_init (MetaStageX11Nested *stage_x11_nested)
+{
+}
+
+static void
+meta_stage_x11_nested_class_init (MetaStageX11NestedClass *klass)
+{
+}
diff --git a/src/backends/x11/meta-stage-x11-nested.h b/src/backends/x11/meta-stage-x11-nested.h
new file mode 100644
index 0000000..412aea0
--- /dev/null
+++ b/src/backends/x11/meta-stage-x11-nested.h
@@ -0,0 +1,34 @@
+/* -*- mode: C; c-file-style: "gnu"; indent-tabs-mode: nil; -*- */
+
+/*
+ * Copyright (C) 2016 Red Hat Inc.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
+ * 02111-1307, USA.
+ *
+ * Written by:
+ * Jonas Ã…dahl <jadahl gmail com>
+ */
+
+#ifndef META_STAGE_X11_NESTED_H
+#define META_STAGE_X11_NESTED_H
+
+#include "clutter/clutter-mutter.h"
+
+#define META_TYPE_STAGE_X11_NESTED (meta_stage_x11_nested_get_type ())
+G_DECLARE_FINAL_TYPE (MetaStageX11Nested, meta_stage_x11_nested,
+ META, STAGE_X11_NESTED, ClutterStageX11)
+
+#endif /* META_STAGE_X11_NESTED_H */
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]