WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
141962
[GTK] Layout Test http/tests/notifications/legacy/notification-request-permission-then-navigate.html is failing
https://bugs.webkit.org/show_bug.cgi?id=141962
Summary
[GTK] Layout Test http/tests/notifications/legacy/notification-request-permis...
Marcos Chavarría Teijeiro (irc: chavaone)
Reported
2015-02-24 01:47:11 PST
The http/tests/notifications/legacy/notification-request-permission-then-navigate.html layout test is since
r177073
(
https://trac.webkit.org/changeset/r177073
). This revision implements support for HTML5 notifications. The diff is the following: --- /home/ch01/wk-tools/layout-test-results/http/tests/notifications/legacy/notification-request-permission-then-navigate-expected.txt +++ /home/ch01/wk-tools/layout-test-results/http/tests/notifications/legacy/notification-request-permission-then-navigate-actual.txt @@ -1,4 +1,5 @@ main frame - has 1 onunload handler(s) +CONSOLE MESSAGE: line 5: TypeError: undefined is not an object (evaluating 'window.webkitNotifications.requestPermission') Tests that we are not crashing when a permission request is answered after navigating On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
Attachments
Patch
(2.06 KB, patch)
2015-03-19 05:11 PDT
,
Marcos Chavarría Teijeiro (irc: chavaone)
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Marcos Chavarría Teijeiro (irc: chavaone)
Comment 1
2015-03-19 05:11:06 PDT
Created
attachment 249032
[details]
Patch
Carlos Garcia Campos
Comment 2
2015-03-19 05:40:12 PDT
Comment on
attachment 249032
[details]
Patch Isn't this a gardening patch? does it need to be reviewed?
WebKit Commit Bot
Comment 3
2015-03-19 06:26:02 PDT
Comment on
attachment 249032
[details]
Patch Clearing flags on attachment: 249032 Committed
r181746
: <
http://trac.webkit.org/changeset/181746
>
WebKit Commit Bot
Comment 4
2015-03-19 06:26:06 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug