Bug 108190
| Summary: | [Qt] New fast/notifications/ introduced in r140927 fails on Qt. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ádám Kallai <kadam> |
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED INVALID | ||
| Severity: | Normal | CC: | abarth, jbadics, jochen, szledan, zarvai |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 87008 | ||
Ádám Kallai
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
These tests are failing only Qt Wk1.
jochen
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
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
Skipped: http://trac.webkit.org/changeset/141112
Jocelyn Turcotte
=== 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.