RESOLVED INVALID 108190
[Qt] New fast/notifications/ introduced in r140927 fails on Qt. 
https://bugs.webkit.org/show_bug.cgi?id=108190
Summary [Qt] New fast/notifications/ introduced in r140927 fails on Qt.
Ádám Kallai
Reported 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?
Attachments
Add attachment proposed patch, testcase, etc.
Ádám Kallai
Comment 1 2013-01-29 07:08:48 PST
These tests are failing only Qt Wk1.
jochen
Comment 2 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
Ádám Kallai
Comment 3 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.
Ádám Kallai
Comment 4 2013-01-29 08:09:12 PST
Skipped: http://trac.webkit.org/changeset/141112
Jocelyn Turcotte
Comment 5 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.
Note You need to log in before you can comment on or make changes to this bug.