Bug 95535 - [Tests] Move fast/notifications/notifications-event-listener-crash.html to http/tests/notifications
Summary: [Tests] Move fast/notifications/notifications-event-listener-crash.html to ht...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.8
: P2 Normal
Assignee: Jon Lee
URL:
Keywords: InRadar
Depends on:
Blocks: 81697
  Show dependency tree / graph
 
Reported: 2012-08-31 01:02 PDT by Jon Lee
Modified: 2012-09-01 00:37 PDT (History)
4 users (show)

See Also:

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 an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jon Lee 2012-08-31 01:02:25 PDT 
Move fast/notifications/notifications-event-listener-crash.html to http/tests/notifications
Comment 1 Radar WebKit Bug Importer 2012-08-31 01:02:54 PDT 
<rdar://problem/12213781>
Comment 2 Jon Lee 2012-08-31 13:42:33 PDT 
Created attachment 161763 [details]
Patch
Comment 3 Jessie Berlin 2012-08-31 13:49:40 PDT 
Why is this move necessary? Why do these tests need to be http tests?
Comment 4 Jessie Berlin 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.
Comment 5 Jon Lee 2012-08-31 14:11:56 PDT 
Created attachment 161772 [details]
Patch
Comment 6 Jon Lee 2012-09-01 00:37:42 PDT 
Committed r127362: <http://trac.webkit.org/changeset/127362>