Merge lp:~larsu/gtk/black-flicker-part2 into lp:~ubuntu-desktop/gtk/ubuntugtk3

Proposed by Lars Karlitski
Status: Merged
Approved by: Didier Roche-Tolomelli
Approved revision: 488
Merged at revision: 488
Proposed branch: lp:~larsu/gtk/black-flicker-part2
Merge into: lp:~ubuntu-desktop/gtk/ubuntugtk3
Diff against target: 95 lines (+36/-34)
4 files modified
debian/changelog (+8/-0)
debian/patches/0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch (+0/-33)
debian/patches/0001-gtkwindow-set-transparent-background-color.patch (+27/-0)
debian/patches/series (+1/-1)
To merge this branch: bzr merge lp:~larsu/gtk/black-flicker-part2
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Review via email: mp+278432@code.launchpad.net

Commit message

Replace d/p/0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch with d/p/0001-gtkwindow-set-transparent-background-color.patch: the old fix broke client side shadow rendering on GNOME shell (LP: #1515561)

Description of the change

Replace d/p/0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch with d/p/0001-gtkwindow-set-transparent-background-color.patch: the old fix broke client side shadow rendering on GNOME shell (LP: #1515561)

To post a comment you must log in.
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Looking good, building fine and tests seems ok! Sponsoring, thanks :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2015-11-17 14:21:53 +0000
3+++ debian/changelog 2015-11-24 11:18:31 +0000
4@@ -1,3 +1,11 @@
5+gtk+3.0 (3.18.5-1ubuntu2) UNRELEASED; urgency=medium
6+
7+ * Replace d/p/0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch
8+ with d/p/0001-gtkwindow-set-transparent-background-color.patch:
9+ - the old fix broke client side shadow rendering on GNOME shell (LP: #1515561)
10+
11+ -- Lars Uebernickel <lars.uebernickel@ubuntu.com> Tue, 24 Nov 2015 12:13:37 +0100
12+
13 gtk+3.0 (3.18.5-1ubuntu1) xenial; urgency=medium
14
15 [ Iain Lane ]
16
17=== removed file 'debian/patches/0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch'
18--- debian/patches/0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch 2015-11-16 16:35:19 +0000
19+++ debian/patches/0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch 1970-01-01 00:00:00 +0000
20@@ -1,33 +0,0 @@
21-From e7528ab9b8ddbad892d9d5ce323549086897b72a Mon Sep 17 00:00:00 2001
22-From: Lars Uebernickel <lars.uebernickel@canonical.com>
23-Date: Mon, 16 Nov 2015 17:29:50 +0100
24-Subject: [PATCH] gtkwindow: continue calling
25- gtk_style_context_set_background()
26-
27-Not doing so introduces black flicker when creating a new window on
28-compiz, because it lacks support for the frame sync protocol.
29-
30-https://bugzilla.gnome.org/show_bug.cgi?id=748498
31----
32- gtk/gtkwindow.c | 5 +++++
33- 1 file changed, 5 insertions(+)
34-
35-diff --git a/gtk/gtkwindow.c b/gtk/gtkwindow.c
36-index 2d9f4b5..0476b6b 100644
37---- a/gtk/gtkwindow.c
38-+++ b/gtk/gtkwindow.c
39-@@ -7122,6 +7122,11 @@ gtk_window_realize (GtkWidget *widget)
40- gtk_widget_register_window (widget, gdk_window);
41- gtk_widget_set_realized (widget, TRUE);
42-
43-+ G_GNUC_BEGIN_IGNORE_DEPRECATIONS
44-+ if (!gtk_widget_get_app_paintable (widget))
45-+ gtk_style_context_set_background (gtk_widget_get_style_context (widget), gdk_window);
46-+ G_GNUC_END_IGNORE_DEPRECATIONS
47-+
48- attributes.x = allocation.x;
49- attributes.y = allocation.y;
50- attributes.width = allocation.width;
51---
52-2.6.2
53-
54
55=== added file 'debian/patches/0001-gtkwindow-set-transparent-background-color.patch'
56--- debian/patches/0001-gtkwindow-set-transparent-background-color.patch 1970-01-01 00:00:00 +0000
57+++ debian/patches/0001-gtkwindow-set-transparent-background-color.patch 2015-11-24 11:18:31 +0000
58@@ -0,0 +1,27 @@
59+From 7bb940fd507ae31e5a06509b1833dd7134ff8b4e Mon Sep 17 00:00:00 2001
60+From: Lars Uebernickel <lars.uebernickel@canonical.com>
61+Date: Tue, 24 Nov 2015 12:08:46 +0100
62+Subject: [PATCH] gtkwindow: set transparent background color
63+
64+To reduce black flickering when new windows are created on compiz.
65+---
66+ gtk/gtkwindow.c | 3 +++
67+ 1 file changed, 3 insertions(+)
68+
69+diff --git a/gtk/gtkwindow.c b/gtk/gtkwindow.c
70+index 2d9f4b5..460a199 100644
71+--- a/gtk/gtkwindow.c
72++++ b/gtk/gtkwindow.c
73+@@ -7122,6 +7122,9 @@ gtk_window_realize (GtkWidget *widget)
74+ gtk_widget_register_window (widget, gdk_window);
75+ gtk_widget_set_realized (widget, TRUE);
76+
77++ GdkRGBA transparent = { 1.0, 1.0, 1.0, 0.0 };
78++ gdk_window_set_background_rgba (gdk_window, &transparent);
79++
80+ attributes.x = allocation.x;
81+ attributes.y = allocation.y;
82+ attributes.width = allocation.width;
83+--
84+2.6.2
85+
86
87=== modified file 'debian/patches/series'
88--- debian/patches/series 2015-11-16 16:35:19 +0000
89+++ debian/patches/series 2015-11-24 11:18:31 +0000
90@@ -23,4 +23,4 @@
91 git_icon_fallback.patch
92 unity_rbga_tooltips.patch
93 git-refresh-mir-backend.patch
94-0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch
95+0001-gtkwindow-set-transparent-background-color.patch

Subscribers

People subscribed via source and target branches

to all changes: