Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#4101 closed defect (fixed)

Source View: Trace when we try to 'set_parent_tansient_for'

Reported by: erikos Owned by: erikos
Priority: Normal Milestone:
Component: Sugar Version: 0.97.x
Severity: Major Keywords: 13.1.0, r+, olpc-test-passed
Cc: humitos Distribution/OS: OLPC
Bug Status: Assigned

Description

Trace:

Traceback (most recent call last):
  File "/home/erikos/sugar-build/install/lib/python2.7/site-packages/jarabe/view/viewsource.py", line 208, in __realize_cb
    self.set_transient_for(parent)
  File "/home/erikos/sugar-build/install/lib/python2.7/site-packages/gi/types.py", line 47, in function
    return info.invoke(*args, **kwargs)
TypeError: argument parent: Expected Gtk.Window, but got gi.repository.GdkX11.X11Window

Previously we did:

>>> import gtk
>>> w = gtk.Window()
>>> gtk.gdk.window_foreign_new(w.window.xid)
<gtk.gdk.Window object at 0xb7699a7c (GdkWindow at 0x8608b10)>

Now we do:

>>> from gi.repository import GdkX11
>>> from gi.repository import Gdk
>>> display = Gdk.Display.get_default()
>>> GdkX11.X11Window.foreign_new_for_display(display, 39845891L)
<X11Window object at 0xb7660a04 (GdkX11Window at 0x8c511f8)>

Attachments (1)

0001-Proper-set-of-set_transient_for-window-SL-4101.patch (1.1 KB) - added by humitos 8 years ago.
A proposal patch to avoid the Traceback

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by humitos

A proposal patch to avoid the Traceback

comment:1 Changed 8 years ago by humitos

  • Keywords r? added

comment:2 Changed 8 years ago by humitos

  • Cc humitos added

comment:3 Changed 8 years ago by erikos

Please be more verbose on the fix in the patch.

comment:4 Changed 8 years ago by erikos

  • Keywords 13.1.0 r+ olpc-test-pending added; r? removed
  • Resolution set to fixed
  • Status changed from new to closed

comment:5 Changed 8 years ago by greenfeld

  • Keywords olpc-test-passed added; olpc-test-pending removed

I do not see this traceback in the view source dialog in OLPC 13.1.0 os30

comment:6 Changed 8 years ago by dnarvaez

  • Milestone 0.98 deleted

Milestone 0.98 deleted

Note: See TracTickets for help on using tickets.