<br><br><div class="gmail_quote">On Tue, Mar 13, 2012 at 2:42 PM, David Levin <span dir="ltr"><<a href="mailto:levin@chromium.org">levin@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Will LEGACY_NOTIFICATIONS cover everything that is in there right now?</blockquote><div><br></div><div>By in there, I meant in WebKit and what hosts have been shipping.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div><br></div><div>Hopefully host apps won't have to define both NOTIFICATIONS and LEGACY_NOTIFICATIONS to keep their current functionality since NOTIFICATIONS sounds like it will be guarding work that is in progress.</div>


<div><br></div><div>dave</div><div class="HOEnZb"><div class="h5"><div><div><br><br><div class="gmail_quote">On Tue, Mar 13, 2012 at 2:40 PM, Jon Lee <span dir="ltr"><<a href="mailto:jonlee@apple.com" target="_blank">jonlee@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">It should reflect whatever is in the notification spec. In the end, when everyone has migrated to the spec, we should be able to get rid of all the #if ENABLE(LEGACY_NOTIFICATIONS) blocks. So LEGACY_NOTIFICATIONS should isolate aspects of notifications that are either replaced by a newer API, or have been removed altogether from the spec.<span><font color="#888888"><div>


<br></div></font></span><div><span><font color="#888888">Jon</font></span><div><div><br><div><br><div><div>On Mar 13, 2012, at 1:38 PM, Jian Li <<a href="mailto:jianli@chromium.org" target="_blank">jianli@chromium.org</a>> wrote:</div>


<br><blockquote type="cite"><div><span>What</span><span> will </span><span>NOTIFICATIONS cover after </span><span>LEGACY_NOTIFICATIONS is being added? Does it cover new syntax only or any syntax that are not considered old?</span></div>




<div><span><br></span></div><div><span>Jian</span></div><div><span><br></span></div><br><div class="gmail_quote">On Tue, Mar 13, 2012 at 1:29 PM, Jon Lee <span dir="ltr"><<a href="mailto:jonlee@apple.com" target="_blank">jonlee@apple.com</a>></span> wrote:<br>




<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">LEGACY_NOTIFICATIONS, for the most part, is exactly what NOTIFICATIONS covers now. So yes, it includes HTML notifications and old syntax, and will not remove anything that already exists.<span><font><div>




<br></div><div>Jon</div></font></span><div><div><div><br><div><div>On Mar 13, 2012, at 1:25 PM, Jian Li <<a href="mailto:jianli@chromium.org" target="_blank">jianli@chromium.org</a>> wrote:</div><br><blockquote type="cite">




Jon, could you please provide what are going to be included in <span>LEGACY_NOTIFICATIONS? </span><font face="arial, sans-serif">Does </font><span>LEGACY_NOTIFICATION</span> only includes HTML notification and old syntax we're considering to deprecate?<div>






<br></div><div>Jian<br><div><div><br><br><div class="gmail_quote">On Mon, Mar 12, 2012 at 7:11 PM, Adam Barth <span dir="ltr"><<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>></span> wrote:<br>




<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

That sounds like a good approach.  Chromium will likely need to<br>
remember to disable NOTIFICATIONS on any upcoming release branches<br>
(until the work is complete).<br>
<span><font><br>
Adam<br>
</font></span><div><div><br>
<br>
On Mon, Mar 12, 2012 at 6:58 PM, Jon Lee <<a href="mailto:jonlee@apple.com" target="_blank">jonlee@apple.com</a>> wrote:<br>
> Hi WebKit!<br>
><br>
> In order to ease the migration path for the nascent notifications API, I'd like to separate the current dependency between NOTIFICATION and LEGACY_NOTIFICATIONS. Currently, in order to support the legacy API, both defines are needed, but ends up also including the new API.<br>







><br>
> Since the future is to eventually move to the spec'd API, I like to separate the two defines, so that NOTIFICATIONS covers the new API, and LEGACY_NOTIFICATIONS the previous one. Currently all ports that support notifications will support both.<br>







><br>
> <<a href="https://bugs.webkit.org/show_bug.cgi?id=80922" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=80922</a>> tracks the work, and once the patch lands,<br>
> ports that wish to avoid exposing the new API should remove the NOTIFICATION define.<br>
><br>
> Any concerns?<br>
><br>
> Thanks,<br>
> Jon<br>
> _______________________________________________<br>
> webkit-dev mailing list<br>
> <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
> <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
</div></div></blockquote></div><br></div></div></div>
</blockquote></div><br></div></div></div></div>
</blockquote></div><br>
</blockquote></div><br></div></div></div></div></div>
<br>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
<br></blockquote></div><br></div></div>
</div></div></blockquote></div><br>