[webkit-dev] Web Notifications update

Maciej Stachowiak mjs at apple.com
Tue Mar 6 22:00:56 PST 2012


On Mar 6, 2012, at 9:00 PM, Jon Lee wrote:

> Whoops, I forgot to mention this also:
> 
> I would like to add a new ENABLE(LEGACY_NOTIFICATION_DEPRECATION) flag which allows ports to deprecate the legacy API. This saves us from the hazards of refactoring the code to use a different ENABLE flag for the legacy API, and allows ports to opt-in when they're ready. Does this sound good?

It's thoughtful to do it that way, but I think we should stick with the standard approach of ENABLE flags enabling features, instead of disabling them. So ENABLE(LEGACY_NOTIFICATIONS) and we can default it to on for ports that already had ENABLE(NOTIFICATIONS) historically. 

Regards,
Maciej



More information about the webkit-dev mailing list