Bug 63244 - [Qt] Fix tst_QWebPage::showModalDialog() API test
Summary: [Qt] Fix tst_QWebPage::showModalDialog() API test
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Jocelyn Turcotte
Keywords: Qt, QtTriaged
Depends on:
Blocks: 38654
  Show dependency treegraph
Reported: 2011-06-23 04:28 PDT by Csaba Osztrogonác
Modified: 2012-11-12 07:31 PST (History)
4 users (show)

See Also:

Remove QEXPECT_FAIL for tests marked as XPASS. (2.22 KB, patch)
2012-07-16 04:15 PDT, Sergio Ahumada
no flags Details | Formatted Diff | Diff
Patch (1.53 KB, patch)
2012-11-09 08:54 PST, Jocelyn Turcotte
hausmann: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2011-06-23 04:28:37 PDT

FAIL!  : tst_QWebPage::showModalDialog() Compared values are not the same
   Actual (res): 
   Expected (QString("This is a test")): This is a test
   Loc: [/ramdisk/qt-linux-release/build/Source/WebKit/qt/tests/qwebpage/tst_qwebpage.cpp(2796)]
Comment 1 Csaba Osztrogonác 2011-06-23 05:50:20 PDT
Marked as expected fail until fix: http://trac.webkit.org/changeset/89560
Comment 2 Sergio Ahumada 2012-02-09 14:20:33 PST
This test is passing in Qt Project CI

2/10/12 7:06:24 AM EST: XPASS  : tst_QWebPage::showModalDialog() COMPARE()
2/10/12 7:06:24 AM EST:    Loc: [../../../src/3rdparty/webkit/Source/WebKit/qt/tests/qwebpage/tst_qwebpage.cpp(2830)]
Comment 3 Sergio Ahumada 2012-07-16 04:15:11 PDT
Created attachment 152507 [details]
Remove QEXPECT_FAIL for tests marked as XPASS.

This patch also applies to https://bugs.webkit.org/show_bug.cgi?id=63245
Comment 4 Simon Hausmann 2012-07-24 00:23:43 PDT
Comment on attachment 152507 [details]
Remove QEXPECT_FAIL for tests marked as XPASS.

Hmm, in principle this sounds okay to me. But the patch doesn't apply, it uses src/3rdparty/webkit as prefix. Did you make this patch against Qt 4.8? It sounds like it needs to be rebased against WebKit trunk.

The patch is also missing a ChangeLog entry.
Comment 5 Jocelyn Turcotte 2012-11-09 08:54:53 PST
Created attachment 173315 [details]
Comment 6 Simon Hausmann 2012-11-10 06:01:39 PST
Comment on attachment 173315 [details]

Makes you wonder how this ever passed :)
Comment 7 Jocelyn Turcotte 2012-11-12 07:31:31 PST
Committed r134234: <http://trac.webkit.org/changeset/134234>