Ticket #1373: sugar-1373.patch

bin/sugar-session

@@ -19 +19 @@
 import os
 import sys
 import time
+import subprocess
 
 if os.environ.get('SUGAR_LOGGER_LEVEL', '') == 'debug':
     print '%r STARTUP: Starting the shell' % time.time()
@@ -31 +32 @@
 import gtk
 import gobject
 import dbus.glib
+import wnck
 
 try:
     import xklavier
@@ -136 +138 @@
     from jarabe.model import filetransfer
     filetransfer.init()
 
-def setup_keyboard_cb(gconf_client):
+def setup_keyboard_cb():
     logging.debug('STARTUP: setup_keyboard_cb')
+
+    gconf_client = gconf.client_get_default()
+
     try:
         display = gtk.gdk.display_get_default()
         if display is not None:
@@ -176 +181 @@
     except Exception:
         logging.exception('Error during keyboard configuration')
 
+def setup_wm():
+    logging.debug('STARTUP: wm')
+
+    # have to reset cursor(metacity sets it on startup)
+    if subprocess.call('echo $DISPLAY; xsetroot -cursor_name left_ptr', shell=True):
+        logging.warning('Can not reset cursor')
+
+    if subprocess.call('metacity-message disable-keybindings',
+            shell=True):
+        logging.warning('Can not disable metacity keybindings')
+
+def bootstrap():
+    setup_wm()
+    launcher.setup()
+
+    gobject.idle_add(unfreeze_dcon_cb)
+    gobject.idle_add(setup_frame_cb)
+    gobject.idle_add(setup_keyhandler_cb)
+    gobject.idle_add(setup_journal_cb)
+    gobject.idle_add(setup_notification_service_cb)
+    gobject.idle_add(setup_file_transfer_cb)
+    gobject.idle_add(show_software_updates_cb)
+
+    if sys.modules.has_key('xklavier'):
+        gobject.idle_add(setup_keyboard_cb)
+
 def main():
     cleanup_logs()
     logger.start('shell')
@@ -200 +231 @@
     icons_path = os.path.join(config.data_path, 'icons')
     gtk.icon_theme_get_default().append_search_path(icons_path)
 
-    launcher.setup()
-
+    # open homewindow before wm to let desktop appear fast
     home_window = homewindow.get_instance()
     home_window.show()
 
-    gobject.idle_add(unfreeze_dcon_cb)
-    gobject.idle_add(setup_frame_cb)
-    gobject.idle_add(setup_keyhandler_cb)
-    gobject.idle_add(setup_journal_cb)
-    gobject.idle_add(setup_notification_service_cb)
-    gobject.idle_add(setup_file_transfer_cb)
-    gobject.idle_add(show_software_updates_cb)
-    
-    if sys.modules.has_key('xklavier'):
-        gobject.idle_add(setup_keyboard_cb, client)
+    screen = wnck.screen_get_default()
+    screen.connect('window-manager-changed', __window_manager_changed_cb)
 
     try:
         gtk.main()
     except KeyboardInterrupt:
         print 'Ctrl+C pressed, exiting...'
 
+def __window_manager_changed_cb(screen):
+    wm_name = screen.get_window_manager_name()
+    if wm_name is not None:
+        screen.disconnect_by_func(__window_manager_changed_cb)
+        bootstrap()
+
 main()

src/jarabe/model/shell.py

@@ -17 +17 @@
 
 import logging
 import time
-import subprocess
 
 import gconf
 import wnck
@@ -323 +322 @@
         self._screen.connect('window-closed', self._window_closed_cb)
         self._screen.connect('active-window-changed',
                              self._active_window_changed_cb)
-        self._screen.connect('window-manager-changed',
-                             self.__window_manager_changed_cb)
 
         self.zoom_level_changed = dispatch.Signal()
 
@@ -594 +591 @@
             self.notify_launch_failed(activity_id)
         return False
 
-    def __window_manager_changed_cb(self, screen):
-        wm_name = screen.get_window_manager_name()
-        if wm_name is not None:
-            logging.debug('Setup %s wm', wm_name)
-            self._setup_wm()
-
-    def _setup_wm(self):
-        # have to reset cursor(metacity sets it on startup)
-        if subprocess.call('xsetroot -cursor_name left_ptr', shell=True):
-            logging.warning('Can not reset cursor')
-
-        if subprocess.call('metacity-message disable-keybindings',
-                shell=True):
-            logging.warning('Can not disable metacity keybindings')
-
 
 _model = None