diff options
author | Tomeu Vizoso <tomeu@tomeuvizoso.net> | 2008-09-15 10:43:21 (GMT) |
---|---|---|
committer | Tomeu Vizoso <tomeu@tomeuvizoso.net> | 2008-09-15 10:43:21 (GMT) |
commit | 99ffb163d9af204b232e7a5b34c8b61628178b27 (patch) | |
tree | bf38430ec1d243d3174af7e5619bee44a77005df /src | |
parent | c46209c7f5c2a13c48d603ce9ca61f08f36eb3a4 (diff) |
Improve error reporting in the zoom level switching code
Diffstat (limited to 'src')
-rw-r--r-- | src/view/Shell.py | 20 |
1 files changed, 12 insertions, 8 deletions
diff --git a/src/view/Shell.py b/src/view/Shell.py index dbc1546..33dcbc0 100644 --- a/src/view/Shell.py +++ b/src/view/Shell.py @@ -125,12 +125,14 @@ class Shell(gobject.GObject): del self._hosts[xid] def _get_host_from_activity_model(self, activity_model): - host = None - if activity_model is not None: - xid = activity_model.get_xid() - if xid: - host = self._hosts[activity_model.get_xid()] - return host + if activity_model is None: + raise ValueError('activity_model cannot be None') + xid = activity_model.get_xid() + if xid: + return self._hosts[activity_model.get_xid()] + else: + logging.debug('Activity %r dont have a window yet' % activity_model) + return None def get_model(self): return self._model @@ -180,14 +182,16 @@ class Shell(gobject.GObject): def set_zoom_level(self, level): if level == self._model.get_zoom_level(): + logging.debug('Already in the level %r' % level) return self.take_activity_screenshot() if level == shellmodel.ShellModel.ZOOM_ACTIVITY: host = self.get_current_activity() - if host is not None: - host.present() + if host is None: + raise ValueError('No current activity') + host.present() else: self._model.set_zoom_level(level) self._screen.toggle_showing_desktop(True) |