RESOLVED FIXED 37667
Active notifications tests where implemented 
https://bugs.webkit.org/show_bug.cgi?id=37667
Summary Active notifications tests where implemented
John Gregg
Reported 2010-04-15 12:51:18 PDT
Active notifications tests where implemented
Attachments
Patch (23.21 KB, patch)
2010-04-15 13:50 PDT, John Gregg 		no flags
DetailsFormatted DiffDiff
Patch (23.85 KB, patch)
2010-04-19 11:29 PDT, John Gregg 		levin: review+
levin: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
John Gregg
Comment 1 2010-04-15 13:50:32 PDT
Created attachment 53470 [details] Patch
David Levin
Comment 2 2010-04-19 11:01:03 PDT
Comment on attachment 53470 [details] Patch It looks like you are changing some file names basically but you didn't use "svn mv" which is preferred for a number of reasons (including history preservation and better diff's (I think)). So r- due to lack of use of svn mv.
John Gregg
Comment 3 2010-04-19 11:29:10 PDT
Created attachment 53693 [details] Patch
John Gregg
Comment 4 2010-04-19 11:31:37 PDT
Thought i had used svn mv the first time; in any case I reverted and did it again with svn mv, although it still created a bulky diff. johnnyg-macbookpro:notifications johnnyg$ svn mv notifications-check-permission.html-disabled notifications-check-permission.html A notifications-check-permission.html D notifications-check-permission.html-disabled johnnyg-macbookpro:notifications johnnyg$ svn mv notifications-display-close-events.html-disabled notifications-display-close-events.html A notifications-display-close-events.html D notifications-display-close-events.html-disabled johnnyg-macbookpro:notifications johnnyg$ svn mv notifications-double-show.html-disabled notifications-double-show.html A notifications-double-show.html D notifications-double-show.html-disabled johnnyg-macbookpro:notifications johnnyg$ svn mv notifications-no-icon.html-disabled notifications-no-icon.html A notifications-no-icon.html D notifications-no-icon.html-disabled svjohnnyg-macbookpro:notifications johnnyg$ svn mv notifications-request-permission.html-disabled notifications-request-permission.html A notifications-request-permission.html D notifications-request-permission.html-disabled svn johnnyg-macbookpro:notifications johnnyg$ svn mv notifications-with-permission.html-disabled notifications-with-permission.html A notifications-with-permission.html D notifications-with-permission.html-disabled johnnyg-macbookpro:notifications johnnyg$ svn mv notifications-without-permission.html-disabled notifications-without-permission.html A notifications-without-permission.html D notifications-without-permission.html-disabled johnnyg-macbookpro:notifications johnnyg$ svn status D notifications-with-permission.html-disabled D notifications-no-icon.html-disabled D notifications-check-permission.html-disabled A + notifications-display-close-events.html A + notifications-request-permission.html D notifications-display-close-events.html-disabled D notifications-request-permission.html-disabled A + notifications-double-show.html A + notifications-without-permission.html D notifications-double-show.html-disabled D notifications-without-permission.html-disabled A + notifications-with-permission.html A + notifications-no-icon.html A + notifications-check-permission.html
David Levin
Comment 5 2010-04-19 13:35:32 PDT
Comment on attachment 53693 [details] Patch > Index: LayoutTests/ChangeLog > =================================================================== > --- LayoutTests/ChangeLog (revision 57659) > +++ LayoutTests/ChangeLog (working copy) > @@ -1,3 +1,29 @@ > +2010-04-15 John Gregg <johnnyg@google.com> > + > + Reviewed by NOBODY (OOPS!). > + > + Enable the notifications layout tests, but skip them on the ports that don't > + implement notifications. Please add a bug link here. btw, these lines are different from before " --- LayoutTests/fast/notifications/notifications-without-permission.html (revision 57650) (from LayoutTests/fast/notifications/notifications-without-permission.html-disabled:57650)" I'm pretty sure there wasn't a "from" in the previous patch. This clearly indicates that you did svn mv.
John Gregg
Comment 6 2010-04-21 16:16:58 PDT
Committed r58021: <http://trac.webkit.org/changeset/58021>
Note You need to log in before you can comment on or make changes to this bug.