Merge lp:~larsu/notify-osd/fix-1189281 into lp:notify-osd

Proposed by Lars Karlitski
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 466
Merged at revision: 466
Proposed branch: lp:~larsu/notify-osd/fix-1189281
Merge into: lp:notify-osd
Diff against target: 74 lines (+12/-6)
2 files modified
src/display.c (+6/-3)
src/stack.c (+6/-3)
To merge this branch: bzr merge lp:~larsu/notify-osd/fix-1189281
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Mirco Müller (community) Approve
Sebastien Bacher Pending
Review via email: mp+168550@code.launchpad.net

Commit message

Don't crash when an application is fullscreen when a notification arrives.

Description of the change

Don't crash when an application is fullscreen when a notification arrives.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:466
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~larsu/notify-osd/fix-1189281/+merge/168550/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/notify-osd-ci/2/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-amd64-ci/1
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-armhf-ci/1
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-i386-ci/1
Coverity artifacts:
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10932.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10933.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10934.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10935.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10936.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10937.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10938.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10939.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10940.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_10941.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_13648.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/31/artifact/results/coverity/CID_13649.html

Click here to trigger a rebuild:
http://s-jenkins:8080/job/notify-osd-ci/2/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Mirco Müller (macslow) wrote :

Looks ok.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/notify-osd-autolanding/6/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-amd64-autolanding/1
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-armhf-autolanding/1
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-i386-autolanding/1
Coverity artifacts:
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_10932.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_10933.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_10934.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_10935.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_10936.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_10937.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_10938.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_10939.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_10940.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_10941.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_13648.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/33/artifact/results/coverity/CID_13649.html

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:466
http://jenkins.qa.ubuntu.com/job/notify-osd-ci/3/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-amd64-ci/2
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-armhf-ci/2
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notify-osd-saucy-i386-ci/2
Coverity artifacts:
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_10932.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_10933.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_10934.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_10935.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_10936.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_10937.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_10938.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_10939.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_10940.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_10941.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_13648.html
    http://jenkins.qa.ubuntu.com/job/generic-coverity-check/32/artifact/results/coverity/CID_13649.html

Click here to trigger a rebuild:
http://s-jenkins:8080/job/notify-osd-ci/3/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/display.c'
--- src/display.c 2012-08-29 22:05:41 +0000
+++ src/display.c 2013-06-10 21:39:30 +0000
@@ -278,7 +278,8 @@
278 return next_to_display;278 return next_to_display;
279}279}
280280
281static void281/* Returns true if the next bubble is shown */
282static gboolean
282stack_layout (Stack* self)283stack_layout (Stack* self)
283{284{
284 Bubble* bubble = NULL;285 Bubble* bubble = NULL;
@@ -294,7 +295,7 @@
294 /* this actually happens when we're called for a synchronous295 /* this actually happens when we're called for a synchronous
295 bubble or after a bubble timed out, but there where no other296 bubble or after a bubble timed out, but there where no other
296 notifications waiting in the queue */297 notifications waiting in the queue */
297 return;298 return FALSE;
298299
299 if (dnd_dont_disturb_user ()300 if (dnd_dont_disturb_user ()
300 && (! bubble_is_urgent (bubble)))301 && (! bubble_is_urgent (bubble)))
@@ -310,7 +311,7 @@
310 /* loop, in case there are other bubbles to discard */311 /* loop, in case there are other bubbles to discard */
311 stack_layout (self);312 stack_layout (self);
312313
313 return;314 return FALSE;
314 }315 }
315316
316 bubble_set_timeout (bubble,317 bubble_set_timeout (bubble,
@@ -428,4 +429,6 @@
428 bubble_fade_in (bubble, 100);429 bubble_fade_in (bubble, 100);
429 else 430 else
430 bubble_fade_in (bubble, 200);431 bubble_fade_in (bubble, 200);
432
433 return TRUE;
431}434}
432435
=== modified file 'src/stack.c'
--- src/stack.c 2012-08-29 22:05:41 +0000
+++ src/stack.c 2013-06-10 21:39:30 +0000
@@ -575,6 +575,7 @@
575 GdkPixbuf* pixbuf = NULL;575 GdkPixbuf* pixbuf = NULL;
576 gboolean new_bubble = FALSE;576 gboolean new_bubble = FALSE;
577 gboolean turn_into_dialog;577 gboolean turn_into_dialog;
578 guint real_id;
578579
579 // check max. allowed limit queue-size580 // check max. allowed limit queue-size
580 if (g_list_length (self->list) > MAX_STACK_SIZE)581 if (g_list_length (self->list) > MAX_STACK_SIZE)
@@ -660,6 +661,8 @@
660 }661 }
661 }662 }
662663
664 real_id = bubble_get_id (bubble);
665
663 if (hints)666 if (hints)
664 {667 {
665 data = (GValue*) g_hash_table_lookup (hints, "x-canonical-private-synchronous");668 data = (GValue*) g_hash_table_lookup (hints, "x-canonical-private-synchronous");
@@ -769,11 +772,11 @@
769772
770 /* update the layout of the stack;773 /* update the layout of the stack;
771 * this will also open the new bubble */774 * this will also open the new bubble */
772 stack_layout (self);775 if (!stack_layout (self))
776 bubble = NULL;
773 }777 }
774778
775 if (bubble)779 dbus_g_method_return (context, real_id);
776 dbus_g_method_return (context, bubble_get_id (bubble));
777780
778 // FIXME: this is a temporary work-around, I do not like at all, until781 // FIXME: this is a temporary work-around, I do not like at all, until
779 // the heavy memory leakage of notify-osd is fully fixed...782 // the heavy memory leakage of notify-osd is fully fixed...

Subscribers

People subscribed via source and target branches