Bug 95537

Summary: [Tests] Move fast/notifications/notifications-double-show.html to http/tests/notifications
Product: WebKit Reporter: Jon Lee <jonlee>
Component: Tools / TestsAssignee: Jon Lee <jonlee>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, dglazkov, jberlin, jianli, webkit-bug-importer, webkit.review.bot
Priority: P2 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+

Jon Lee
Reported 2012-08-31 01:32:55 PDT
Move fast/notifications/notifications-double-show.html to http/tests/notifications
Attachments
Patch (4.57 KB, patch)
2012-08-31 14:00 PDT, Jon Lee 		no flags
DetailsFormatted DiffDiff
Patch (4.54 KB, patch)
2012-09-07 17:12 PDT, Jon Lee 		ap: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2012-08-31 01:33:24 PDT
<rdar://problem/12213860>
Jon Lee
Comment 2 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.
Jon Lee
Comment 3 2012-08-31 14:00:21 PDT
Created attachment 161767 [details] Patch
WebKit Review Bot
Comment 4 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
Jon Lee
Comment 5 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.
Jon Lee
Comment 6 2012-09-07 17:12:07 PDT
Created attachment 162915 [details] Patch
Jon Lee
Comment 7 2012-09-10 11:06:09 PDT
Committed r128079: <http://trac.webkit.org/changeset/128079>
Note You need to log in before you can comment on or make changes to this bug.