Bug 95537

Summary:

[Tests] Move fast/notifications/notifications-double-show.html to http/tests/notifications

Product:

WebKit

Reporter:

Jon Lee <jonlee>

Component:

Tools / Tests

Assignee:

Jon Lee <jonlee>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ap, dglazkov, jberlin, jianli, webkit-bug-importer, webkit.review.bot

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.8

Bug Depends on:

Bug Blocks:

81697

Attachments:

Description	Flags
Patch	none
Patch	ap: review+

Description Jon Lee 2012-08-31 01:32:55 PDT

Move fast/notifications/notifications-double-show.html to http/tests/notifications

Comment 1 Radar WebKit Bug Importer 2012-08-31 01:33:24 PDT

<rdar://problem/12213860>

Comment 2 Jon Lee 2012-08-31 13:59:51 PDT

This only applies to legacy API because standard API  implies a show call, so it's not possible to do a double-show.

Comment 3 Jon Lee 2012-08-31 14:00:21 PDT

Created attachment 161767 [details]
Patch

Comment 4 WebKit Review Bot 2012-08-31 17:02:08 PDT

Comment on attachment 161767 [details]
Patch

Attachment 161767 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/13729067

New failing tests:
http/tests/cache/subresource-expiration-1.html

Comment 5 Jon Lee 2012-09-01 00:35:39 PDT

not sure why the addition of this test would affect tests on chromium, since chromium currently skips these tests.

Comment 6 Jon Lee 2012-09-07 17:12:07 PDT

Created attachment 162915 [details]
Patch

Comment 7 Jon Lee 2012-09-10 11:06:09 PDT

Committed r128079: <http://trac.webkit.org/changeset/128079>