Bug 108609 - [Qt] New inspector-protocol/page/javascriptDialogEvents.html fails on Qt.
Summary: [Qt] New inspector-protocol/page/javascriptDialogEvents.html fails on Qt.
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 87008
  Show dependency treegraph
 
Reported: 2013-02-01 03:08 PST by Ádám Kallai
Modified: 2014-02-03 03:24 PST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ádám Kallai 2013-02-01 03:08:09 PST
This test started to fail r141555.
Diff is here:
http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release/r141567%20(47621)/inspector-protocol/page/javascriptDialogEvents-pretty-diff.html

Could you check it please? Is it a correct result?
Comment 1 Ken Kania 2013-02-01 08:06:01 PST
The test passed on the qt ews bots. Any ideas what might be different about this bot? I'm building locally now to see what might be wrong. I'll keep you posted.
Comment 2 Ken Kania 2013-02-01 10:24:03 PST
The result you see is correct behavior.

Not sure why the ews bot passed, but the QT DRT just prints "CONFIRM:" instead of "CONFIRM NAVIGATION:" for the before unload prompt. This is because ChromeClientQt::runBeforeUnloadConfirmPanel simply delegates to runJavaScriptConfirm, which prints CONFIRM: instead of CONFIRM NAVIGATION:, unlike the other DRTs.

The one other test that depends on the unload confirm panel is disabled on qt too: fast/loader/form-submission-after-beforeunload-cancel.html
See https://bugs.webkit.org/show_bug.cgi?id=66162.
Comment 3 Ádám Kallai 2013-02-06 01:34:01 PST
Yes, there is a difference. The qt ews only build patch. Therefore it could be green in this case. I will add to platform specific expected for this test.
Thank for your response.
Comment 4 Jocelyn Turcotte 2014-02-03 03:24:38 PST
=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.