I see where the confusion is, and I misspoke. You can include either NOTIFICATIONS or LEGACY_NOTIFICATIONS and have a complete API. LEGACY_NOTIFICATIONS should cover all of the previous functionality, and NOTIFICATIONS will cover the new API. Therefore, APIs that are common between the two will have #if ENABLE(NOTIFICATIONS) || ENABLE(LEGACY_NOTIFICATIONS)
The patch I am putting up for review will initially set both to be exactly the same. That is, all #if ENABLE(NOTIFICATIONS) is replaced by #if ENABLE(NOTIFICATIONS) || ENABLE(LEGACY_NOTIFICATIONS) and something similar for the .idl's. As other bugs with patches begin to migrate to the new API, the defines will begin to split. Hope that helps. Sorry about the confusion. Jon On Mar 13, 2012, at 2:42 PM, David Levin <[email protected]> wrote: > Will LEGACY_NOTIFICATIONS cover everything that is in there right now? > > 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. > > dave > > > On Tue, Mar 13, 2012 at 2:40 PM, Jon Lee <[email protected]> wrote: > 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. > > Jon > > > On Mar 13, 2012, at 1:38 PM, Jian Li <[email protected]> wrote: > >> What will NOTIFICATIONS cover after LEGACY_NOTIFICATIONS is being added? >> Does it cover new syntax only or any syntax that are not considered old? >> >> Jian >> >> >> On Tue, Mar 13, 2012 at 1:29 PM, Jon Lee <[email protected]> wrote: >> 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. >> >> Jon >> >> On Mar 13, 2012, at 1:25 PM, Jian Li <[email protected]> wrote: >> >>> Jon, could you please provide what are going to be included in >>> LEGACY_NOTIFICATIONS? Does LEGACY_NOTIFICATION only includes HTML >>> notification and old syntax we're considering to deprecate? >>> >>> Jian >>> >>> >>> On Mon, Mar 12, 2012 at 7:11 PM, Adam Barth <[email protected]> wrote: >>> That sounds like a good approach. Chromium will likely need to >>> remember to disable NOTIFICATIONS on any upcoming release branches >>> (until the work is complete). >>> >>> Adam >>> >>> >>> On Mon, Mar 12, 2012 at 6:58 PM, Jon Lee <[email protected]> wrote: >>> > Hi WebKit! >>> > >>> > 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. >>> > >>> > 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. >>> > >>> > <https://bugs.webkit.org/show_bug.cgi?id=80922> tracks the work, and once >>> > the patch lands, >>> > ports that wish to avoid exposing the new API should remove the >>> > NOTIFICATION define. >>> > >>> > Any concerns? >>> > >>> > Thanks, >>> > Jon >>> > _______________________________________________ >>> > webkit-dev mailing list >>> > [email protected] >>> > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev >>> _______________________________________________ >>> webkit-dev mailing list >>> [email protected] >>> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev >>> >> >> > > > _______________________________________________ > webkit-dev mailing list > [email protected] > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev > >
_______________________________________________ webkit-dev mailing list [email protected] http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
