Bug 959 – can't pass QVariant to the QtWebkit bridge

PySide Bugzilla Closed for New Bugs

PySide is now a Qt Add-on and uses the Qt Project's JIRA Bug Tracker instead of this Bugzilla instance. This Bugzilla is left for reference purposes.

Bug 959 - can't pass QVariant to the QtWebkit bridge

Summary:

can't pass QVariant to the QtWebkit bridge

Status:	REOPENED

Product:	PySide
Classification:	Unclassified
Component:	QtCore
Version:	1.0.5
Platform:	All All

Importance:	P3 normal
Assigned To:	renato filho

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2011-08-07 01:00 EEST by Farsmo
Modified:	2012-04-25 10:16 EEST (History)
CC List:	9 users (show)

See Also:

Attachments
test case (2.18 KB, text/plain) 2011-08-11 09:42 EEST, Farsmo	Details
cpp test (2.80 KB, application/x-bzip) 2011-08-23 23:15 EEST, renato filho	Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Farsmo 2011-08-07 01:00:41 EEST

See the attached test case. Returning QVariants from a slot works fine. But
when passing an argument to a QtWebkit bridge slot:
- everything works fine for simple types like unicode and QStringList
- the slot connection fails for QVariant
- the program freezes with QVariantList and QVariantMap

Comment 1 Hugo Parente Lima 2011-08-10 20:49:41 EEST

The attached test case is missing :-/, could you attach it?

Comment 2 Farsmo 2011-08-11 09:42:57 EEST

Created attachment 396 [details]
test case

That's weird, I'm pretty sure I selected the file when creating the bug. Maybe
I didn't put a description so it was ignored.

Comment 3 Farsmo 2011-08-18 12:17:52 EEST

Updating the severity to "normal" because of the freeze.

Comment 4 renato filho 2011-08-23 23:15:10 EEST

Created attachment 411 [details]
cpp test

Comment 5 renato filho 2011-08-25 22:29:53 EEST

The signal with QVariant does not work in C++ example too, but the other signal
now works fine in both pyside and c++, then I will consider this bug fixed, you
can use my c++ example to report a bug on webkit bugzilla.

fixed on pyside commit:
commit 9ff7b38d9edab6a5e5b23a675a411ce5a053a067
Author: Renato Filho <renato.filho@openbossa.org>
Date:   Wed Aug 24 14:57:24 2011 -0300

Comment 6 Farsmo 2011-08-28 12:11:42 EEST

I'm surprised that adding only QVariantMap also fixes the QVariantList bug. Did
you have a freeze with QVariantList before the patch?

I reported the QVariant bug upstream:
https://bugs.webkit.org/show_bug.cgi?id=67092

Comment 7 Farsmo 2011-08-28 19:14:06 EEST

Never mind, I don't have a QVariantList freeze any more so I guess it was
already fixed.

Comment 8 renato filho 2011-09-21 20:49:19 EEST

Release 1.0.7

Comment 9 Didier Raboud 2012-02-09 18:02:00 EET

Hi,

according to Debian's build logs, it seems that this bug is not fixed for all
architectures in pyside 1.1.0.

Analysis script output:
   
http://alioth.debian.org/~odyx-guest/packages/pyside/pyside_1.1.0-1_analysis.txt

(tl;dr: the QtWebKit_bug_959 fails on the following situations:
     on arch:i386 with python-2.6
    on arch:i386 with python-2.7
    on arch:i386 with python-3.2
    on arch:ia64 with python-2.6
    on arch:ia64 with python-2.7
    on arch:ia64 with python-3.2
    on arch:kfreebsd-amd64 with python-2.6
    on arch:kfreebsd-amd64 with python-2.7
    on arch:kfreebsd-amd64 with python-3.2
    on arch:kfreebsd-i386 with python-2.6
    on arch:kfreebsd-i386 with python-2.7
    on arch:kfreebsd-i386 with python-3.2
    on arch:mipsel with python-2.6
    on arch:mipsel with python-2.7
    on arch:mipsel with python-3.2
    on arch:mips with python-2.6
    on arch:mips with python-2.7
    on arch:mips with python-3.2
    on arch:powerpc with python-2.6
    on arch:powerpc with python-2.7
    on arch:powerpc with python-3.2
    on arch:s390 with python-2.6
    on arch:s390 with python-2.7
    on arch:s390 with python-3.2
    on arch:s390x with python-2.6
    on arch:s390x with python-2.7
    on arch:s390x with python-3.2
    on arch:sparc with python-2.6
    on arch:sparc with python-2.7
    on arch:sparc with python-3.2
)

Running the test on i386 reveals a segmentation fault.

Cheers, 

OdyX

Comment 10 Matti Airas 2012-03-08 16:57:17 EET

PySide is now a Qt-addon and uses Qt Project's JIRA tool for tracking bugs.
Please verify that the bug is still valid and re-submit it in the address
below:

https://bugreports.qt-project.org/

Sorry for the inconvenience!

Comment 11 Didier Raboud 2012-04-25 10:16:21 EEST

This has been reported as https://bugreports.qt-project.org/browse/PYSIDE-61 .

Cheers,

OdyX