RESOLVED FIXED 82121
Notifications should be in Modules/notifications 
https://bugs.webkit.org/show_bug.cgi?id=82121
Summary Notifications should be in Modules/notifications
Adam Barth
Reported 2012-03-24 01:23:26 PDT
Notifications should be in Modules/notifications
Attachments
Patch (120.56 KB, patch)
2012-03-24 01:25 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
Patch for EFL and GTK port (134.88 KB, patch)
2012-06-19 19:42 PDT, Gyuyoung Kim 		gyuyoung.kim: commit-queue-
DetailsFormatted DiffDiff
Patch for EFL ,GTK and QT port (146.84 KB, patch)
2012-06-19 21:37 PDT, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
Patch (204.33 KB, patch)
2012-06-20 11:40 PDT, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
Patch (179.31 KB, patch)
2012-07-03 07:13 PDT, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
Patch (188.07 KB, patch)
2012-07-04 02:42 PDT, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
Show Obsolete (5) View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2012-03-24 01:25:08 PDT
Created attachment 133624 [details] Patch
Early Warning System Bot
Comment 2 2012-03-24 01:37:09 PDT
Comment on attachment 133624 [details] Patch Attachment 133624 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12128047
Gustavo Noronha (kov)
Comment 3 2012-03-24 01:39:44 PDT
Comment on attachment 133624 [details] Patch Attachment 133624 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/12123964
Early Warning System Bot
Comment 4 2012-03-24 01:40:12 PDT
Comment on attachment 133624 [details] Patch Attachment 133624 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/12131011
Build Bot
Comment 5 2012-03-24 01:46:44 PDT
Comment on attachment 133624 [details] Patch Attachment 133624 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12132003
WebKit Review Bot
Comment 6 2012-03-24 01:51:17 PDT
Comment on attachment 133624 [details] Patch Attachment 133624 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12131010
Gyuyoung Kim
Comment 7 2012-03-24 01:52:04 PDT
Comment on attachment 133624 [details] Patch Attachment 133624 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/12131013
Adam Barth
Comment 8 2012-03-24 01:55:12 PDT
The diff is screwed up somehow. I think because I'm adding the whole directory? /me is confused.
Build Bot
Comment 9 2012-03-24 01:59:36 PDT
Comment on attachment 133624 [details] Patch Attachment 133624 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/12123962
Adam Barth
Comment 10 2012-03-26 15:49:13 PDT
Comment on attachment 133624 [details] Patch I need to make a version of this patch that, you know, compiles.
Gyuyoung Kim
Comment 11 2012-06-19 19:42:37 PDT
Created attachment 148488 [details] Patch for EFL and GTK port Hello Adam, I wonder whether you will submit modified patch. Anyway, to help you, I make a patch for EFL and GTK ports. However, if you are busy to make this patch again, I would like to submit a patch in order to support all ports.
Adam Barth
Comment 12 2012-06-19 20:32:50 PDT
Thank you. I've gotten distracted by other patches. If you work up a patch here, I'd be happy to review it.
Gyuyoung Kim
Comment 13 2012-06-19 21:37:41 PDT
Created attachment 148501 [details] Patch for EFL ,GTK and QT port Thank you for your consideration. This patch is for EFL, GTK and QT ports for now. I'm gonna submit a patch which supports Chromium, Mac and Win ports as well.
Build Bot
Comment 14 2012-06-19 21:55:47 PDT
Comment on attachment 148501 [details] Patch for EFL ,GTK and QT port Attachment 148501 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13007092
Build Bot
Comment 15 2012-06-19 22:01:25 PDT
Comment on attachment 148501 [details] Patch for EFL ,GTK and QT port Attachment 148501 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13002101
WebKit Review Bot
Comment 16 2012-06-19 22:06:24 PDT
Comment on attachment 148501 [details] Patch for EFL ,GTK and QT port Attachment 148501 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/13008095
Gyuyoung Kim
Comment 17 2012-06-20 11:40:30 PDT
Created attachment 148614 [details] Patch
Build Bot
Comment 18 2012-06-20 12:00:49 PDT
Comment on attachment 148614 [details] Patch Attachment 148614 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13005271
Build Bot
Comment 19 2012-06-20 12:56:04 PDT
Comment on attachment 148614 [details] Patch Attachment 148614 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12998322
Adam Barth
Comment 20 2012-06-27 13:33:15 PDT
Comment on attachment 148614 [details] Patch This doesn't appear to build on mac or win.
Gyuyoung Kim
Comment 21 2012-07-03 07:13:22 PDT
Created attachment 150603 [details] Patch
Gyuyoung Kim
Comment 22 2012-07-03 07:15:47 PDT
This patch is for all ports except for win port. I'm still working on win port as well.
Build Bot
Comment 23 2012-07-03 07:41:36 PDT
Comment on attachment 150603 [details] Patch Attachment 150603 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13139175
Gyuyoung Kim
Comment 24 2012-07-04 02:42:00 PDT
Created attachment 150747 [details] Patch
Adam Barth
Comment 25 2012-07-04 21:42:53 PDT
Comment on attachment 150747 [details] Patch Thanks for doing this!
WebKit Review Bot
Comment 26 2012-07-04 23:02:58 PDT
Comment on attachment 150747 [details] Patch Clearing flags on attachment: 150747 Committed r121882: <http://trac.webkit.org/changeset/121882>
WebKit Review Bot
Comment 27 2012-07-04 23:03:14 PDT
All reviewed patches have been landed. Closing bug.
Andy Estes
Comment 28 2012-07-06 15:14:37 PDT
I pointed DerivedSource.make to the new location for notifications idl files in <http://trac.webkit.org/changeset/122036>.
Gyuyoung Kim
Comment 29 2012-07-07 18:03:12 PDT
(In reply to comment #28) > I pointed DerivedSource.make to the new location for notifications idl files in <http://trac.webkit.org/changeset/122036>. I'm sorry. I missed it. Thanks.
Note You need to log in before you can comment on or make changes to this bug.