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
=== modified file 'debian/changelog'
--- debian/changelog 2015-11-17 14:21:53 +0000
+++ debian/changelog 2015-11-24 11:18:31 +0000
@@ -1,3 +1,11 @@
1gtk+3.0 (3.18.5-1ubuntu2) UNRELEASED; urgency=medium
2
3 * Replace d/p/0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch
4 with d/p/0001-gtkwindow-set-transparent-background-color.patch:
5 - the old fix broke client side shadow rendering on GNOME shell (LP: #1515561)
6
7 -- Lars Uebernickel <lars.uebernickel@ubuntu.com> Tue, 24 Nov 2015 12:13:37 +0100
8
1gtk+3.0 (3.18.5-1ubuntu1) xenial; urgency=medium9gtk+3.0 (3.18.5-1ubuntu1) xenial; urgency=medium
210
3 [ Iain Lane ]11 [ Iain Lane ]
412
=== removed file 'debian/patches/0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch'
--- debian/patches/0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch 2015-11-16 16:35:19 +0000
+++ debian/patches/0001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch 1970-01-01 00:00:00 +0000
@@ -1,33 +0,0 @@
1From e7528ab9b8ddbad892d9d5ce323549086897b72a Mon Sep 17 00:00:00 2001
2From: Lars Uebernickel <lars.uebernickel@canonical.com>
3Date: Mon, 16 Nov 2015 17:29:50 +0100
4Subject: [PATCH] gtkwindow: continue calling
5 gtk_style_context_set_background()
6
7Not doing so introduces black flicker when creating a new window on
8compiz, because it lacks support for the frame sync protocol.
9
10https://bugzilla.gnome.org/show_bug.cgi?id=748498
11---
12 gtk/gtkwindow.c | 5 +++++
13 1 file changed, 5 insertions(+)
14
15diff --git a/gtk/gtkwindow.c b/gtk/gtkwindow.c
16index 2d9f4b5..0476b6b 100644
17--- a/gtk/gtkwindow.c
18+++ b/gtk/gtkwindow.c
19@@ -7122,6 +7122,11 @@ gtk_window_realize (GtkWidget *widget)
20 gtk_widget_register_window (widget, gdk_window);
21 gtk_widget_set_realized (widget, TRUE);
22
23+ G_GNUC_BEGIN_IGNORE_DEPRECATIONS
24+ if (!gtk_widget_get_app_paintable (widget))
25+ gtk_style_context_set_background (gtk_widget_get_style_context (widget), gdk_window);
26+ G_GNUC_END_IGNORE_DEPRECATIONS
27+
28 attributes.x = allocation.x;
29 attributes.y = allocation.y;
30 attributes.width = allocation.width;
31--
322.6.2
33
340
=== added file 'debian/patches/0001-gtkwindow-set-transparent-background-color.patch'
--- debian/patches/0001-gtkwindow-set-transparent-background-color.patch 1970-01-01 00:00:00 +0000
+++ debian/patches/0001-gtkwindow-set-transparent-background-color.patch 2015-11-24 11:18:31 +0000
@@ -0,0 +1,27 @@
1From 7bb940fd507ae31e5a06509b1833dd7134ff8b4e Mon Sep 17 00:00:00 2001
2From: Lars Uebernickel <lars.uebernickel@canonical.com>
3Date: Tue, 24 Nov 2015 12:08:46 +0100
4Subject: [PATCH] gtkwindow: set transparent background color
5
6To reduce black flickering when new windows are created on compiz.
7---
8 gtk/gtkwindow.c | 3 +++
9 1 file changed, 3 insertions(+)
10
11diff --git a/gtk/gtkwindow.c b/gtk/gtkwindow.c
12index 2d9f4b5..460a199 100644
13--- a/gtk/gtkwindow.c
14+++ b/gtk/gtkwindow.c
15@@ -7122,6 +7122,9 @@ gtk_window_realize (GtkWidget *widget)
16 gtk_widget_register_window (widget, gdk_window);
17 gtk_widget_set_realized (widget, TRUE);
18
19+ GdkRGBA transparent = { 1.0, 1.0, 1.0, 0.0 };
20+ gdk_window_set_background_rgba (gdk_window, &transparent);
21+
22 attributes.x = allocation.x;
23 attributes.y = allocation.y;
24 attributes.width = allocation.width;
25--
262.6.2
27
028
=== modified file 'debian/patches/series'
--- debian/patches/series 2015-11-16 16:35:19 +0000
+++ debian/patches/series 2015-11-24 11:18:31 +0000
@@ -23,4 +23,4 @@
23git_icon_fallback.patch23git_icon_fallback.patch
24unity_rbga_tooltips.patch24unity_rbga_tooltips.patch
25git-refresh-mir-backend.patch25git-refresh-mir-backend.patch
260001-gtkwindow-continue-calling-gtk_style_context_set_bac.patch260001-gtkwindow-set-transparent-background-color.patch

Subscribers

People subscribed via source and target branches

to all changes: