Bug 108190 - [Qt] New fast/notifications/ introduced in r140927 fails on Qt.
Summary: [Qt] New fast/notifications/ introduced in r140927 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 tree / graph
 
Reported: 2013-01-29 07:02 PST by Ádám Kallai
Modified: 2014-02-03 03:24 PST (History)
5 users (show)

See Also:

Attachments
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 Ádám Kallai 2013-01-29 07:02:43 PST 
Two tests started to fail on Qt.

 * fast/notifications/notifications-constructor-with-permission.html
diff is here:
http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release/r141103%20(47405)/fast/notifications/notifications-constructor-with-permission-pretty-diff.html

 * fast/notifications/notifications-constructor-without-permission.html
http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release/r141103%20(47405)/fast/notifications/notifications-constructor-without-permission-pretty-diff.html

I don't have any idea why. Could you check it please?
Comment 1 Ádám Kallai 2013-01-29 07:08:48 PST 
These tests are failing only Qt Wk1.
Comment 2 jochen 2013-01-29 07:10:37 PST 
Does Qt Wk1 enabled notifications (as opposed to legacy notifications)? If not, you should skip the test. If yes, notifications don't work on Qt Wk1
Comment 3 Ádám Kallai 2013-01-29 07:30:53 PST 
Thanks for your response. Yes, notifications are enabled on Qt. Probably They don't work in Qt.
I'm going to add them to Qt TestExpectations.
Comment 4 Ádám Kallai 2013-01-29 08:09:12 PST 
Skipped: http://trac.webkit.org/changeset/141112
Comment 5 Jocelyn Turcotte 2014-02-03 03:24:30 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.