WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
95535
[Tests] Move fast/notifications/notifications-event-listener-crash.html to http/tests/notifications
https://bugs.webkit.org/show_bug.cgi?id=95535
Summary
[Tests] Move fast/notifications/notifications-event-listener-crash.html to ht...
Jon Lee
Reported
2012-08-31 01:02:25 PDT
Move fast/notifications/notifications-event-listener-crash.html to http/tests/notifications
Attachments
Patch
(3.82 KB, patch)
2012-08-31 13:42 PDT
,
Jon Lee
no flags
Details
Formatted Diff
Diff
Patch
(4.29 KB, patch)
2012-08-31 14:11 PDT
,
Jon Lee
jberlin
: review+
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2012-08-31 01:02:54 PDT
<
rdar://problem/12213781
>
Jon Lee
Comment 2
2012-08-31 13:42:33 PDT
Created
attachment 161763
[details]
Patch
Jessie Berlin
Comment 3
2012-08-31 13:49:40 PDT
Why is this move necessary? Why do these tests need to be http tests?
Jessie Berlin
Comment 4
2012-08-31 13:56:08 PDT
Comment on
attachment 161763
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=161763&action=review
> LayoutTests/ChangeLog:8 > +
Please explain why this move is necessary (consolidate the tests in one place, etc).
> LayoutTests/ChangeLog:12 > + * http/tests/notifications/event-listener-crash.html: Added.
You need to mention that you changed this test in addition to moving it, and why.
Jon Lee
Comment 5
2012-08-31 14:11:56 PDT
Created
attachment 161772
[details]
Patch
Jon Lee
Comment 6
2012-09-01 00:37:42 PDT
Committed
r127362
: <
http://trac.webkit.org/changeset/127362
>
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