Merge lp:~gary-lasker/software-center/fix-crash-lp1021308 into lp:software-center

[Gary Lasker]

A branch to fix the crash in bug 1021308. Note that I am unable to reproduce the conditions that cause this crash, but the fix I've implemented should take care of it in a straightforward way for all children of the StatusBar class, and it has no discernable ill effects in my testing.

This one is currently #13 in the list of Quantal's top crashers at errors.ubuntu.com.

Many thanks for your review!

[lp:~gary-lasker/software-center/fix-crash-lp1021308]

3214. By Gary Lasker on 2012-10-03

tweak comment

[Michael Vogt (mvo)]

Thanks for your branch.

I was able to reproduce the error using the instructions given in https://bugs.launchpad.net/ubuntu/+source/software-center/+bug/1021308/comments/3 (actually it was enough to run software-center from the kubuntu live-cd in a VM to trigger it). I also put instructions how to reproduce on a normal ubuntu here: https://bugs.launchpad.net/ubuntu/+source/software-center/+bug/1021308/comments/10

While this fixes the crash at hand there is another crash in the description widget plus the UI is in general in a pretty bad shape as the signal is needed in various other places as well, most noticeable in the applistview. So I think we need to fix all places where a "style-updated" signal is expected not just this one place.

Its probably easiest to simply do something like:
         self.connect("style-updated", self.on_style_updated)
+ # workaround broken engines (LP: #1021308)
+ self.emit("style-updated")

after each connect("style-updated"). Or to fix the engine, but given that lubuntu seems to have a similar problem
I suspect that there are various broken theme engines.

[lp:~gary-lasker/software-center/fix-crash-lp1021308]

3215. By Gary Lasker on 2012-10-05

add mvo's workaround for broken theme engines to every instance of connecting for a style-updated signal

3216. By Gary Lasker on 2012-10-05

pep8 fix

[Gary Lasker (gary-lasker)]

Hi Michael, thank you for the info and for conditions to reproduce, that is extremely helpful. With the oxygen theme I see the crash now (btw, USC is quite unusable using this theme due to the text colors, but that's a separate bug).

I added a self.emit("style-updated") after every instance of "connect('style-updated', ...", per your suggestion. Things seem to be working for me now, but I'd appreciate it if you could look this over.

It's pretty nasty that the theme engines have this issue!

[Gary Lasker (gary-lasker)]

review: Needs Resubmitting

=== modified file 'softwarecenter/ui/gtk3/app.py'
--- softwarecenter/ui/gtk3/app.py	2012-09-20 01:26:42 +0000
+++ softwarecenter/ui/gtk3/app.py	2012-10-05 03:37:22 +0000
@@ -361,6 +361,8 @@
                                  init_sc_css_provider,
                                  settings,
                                  Gdk.Screen.get_default())
+        # workaround broken engines (LP: #1021308)
+        self.window_main.emit("style-updated")
 
         # register view manager and create view panes/widgets
         self.view_manager = ViewManager(self.notebook_view, options)

=== modified file 'softwarecenter/ui/gtk3/views/appdetailsview.py'
--- softwarecenter/ui/gtk3/views/appdetailsview.py	2012-10-01 14:23:31 +0000
+++ softwarecenter/ui/gtk3/views/appdetailsview.py	2012-10-05 03:37:22 +0000
@@ -101,6 +101,8 @@
         self._bg = [1, 1, 1, 0.3]
 
         self.connect("style-updated", self.on_style_updated)
+        # workaround broken engines (LP: #1021308)
+        self.emit("style-updated")
 
     def on_style_updated(self, widget):
         context = self.get_style_context()

=== modified file 'softwarecenter/ui/gtk3/widgets/apptreeview.py'
--- softwarecenter/ui/gtk3/widgets/apptreeview.py	2012-09-18 20:14:56 +0000
+++ softwarecenter/ui/gtk3/widgets/apptreeview.py	2012-10-05 03:37:22 +0000
@@ -104,6 +104,8 @@
         self._cursor_hand = Gdk.Cursor.new(Gdk.CursorType.HAND2)
 
         self.connect("style-updated", self._on_style_updated, self._renderer)
+        # workaround broken engines (LP: #1021308)
+        self.emit("style-updated")
         # button and motion are "special"
         self.connect("button-press-event", self._on_button_press_event,
                      self._renderer)

=== modified file 'softwarecenter/ui/gtk3/widgets/buttons.py'
--- softwarecenter/ui/gtk3/widgets/buttons.py	2012-09-18 20:14:56 +0000
+++ softwarecenter/ui/gtk3/widgets/buttons.py	2012-10-05 03:37:22 +0000
@@ -467,6 +467,8 @@
 
         self.connect('size-allocate', self.on_size_allocate)
         self.connect('style-updated', self.on_style_updated)
+        # workaround broken engines (LP: #1021308)
+        self.emit("style-updated")
 
     def on_size_allocate(self, *args):
         alloc = self.get_allocation()

=== modified file 'softwarecenter/ui/gtk3/widgets/containers.py'
--- softwarecenter/ui/gtk3/widgets/containers.py	2012-09-17 09:04:43 +0000
+++ softwarecenter/ui/gtk3/widgets/containers.py	2012-10-05 03:37:22 +0000
@@ -248,6 +248,8 @@
         self._allocation = Gdk.Rectangle()
         self.connect("size-allocate", self.on_size_allocate)
         self.connect("style-updated", self.on_style_updated)
+        # workaround broken engines (LP: #1021308)
+        self.emit("style-updated")
 
     def on_style_updated(self, widget):
         self._frame_surface_cache = None

=== modified file 'softwarecenter/ui/gtk3/widgets/description.py'
--- softwarecenter/ui/gtk3/widgets/description.py	2012-09-12 14:08:18 +0000
+++ softwarecenter/ui/gtk3/widgets/description.py	2012-10-05 03:37:22 +0000
@@ -448,6 +448,8 @@
 
         self.connect("size-allocate", self.on_size_allocate)
         self.connect('style-updated', self._on_style_updated)
+        # workaround broken engines (LP: #1021308)
+        self.emit("style-updated")
 
     def on_size_allocate(self, *args):
         allocation = self.get_allocation()

=== modified file 'softwarecenter/ui/gtk3/widgets/separators.py'
--- softwarecenter/ui/gtk3/widgets/separators.py	2012-09-12 14:08:18 +0000
+++ softwarecenter/ui/gtk3/widgets/separators.py	2012-10-05 03:37:22 +0000
@@ -8,6 +8,8 @@
         context = self.get_style_context()
         context.add_class("item-view-separator")
         self.connect("style-updated", self.on_style_updated)
+        # workaround broken engines (LP: #1021308)
+        self.emit("style-updated")
         return
 
     def on_style_updated(self, widget):

=== modified file 'softwarecenter/ui/gtk3/widgets/stars.py'
--- softwarecenter/ui/gtk3/widgets/stars.py	2012-05-30 18:39:55 +0000
+++ softwarecenter/ui/gtk3/widgets/stars.py	2012-10-05 03:37:22 +0000
@@ -303,6 +303,8 @@
         self.set_visible_window(False)
         self.connect("draw", self.on_draw)
         self.connect("style-updated", self.on_style_updated)
+        # workaround broken engines (LP: #1021308)
+        self.emit("style-updated")
 
     def do_get_preferred_width(self):
         context = self.get_style_context()