search

sysapps / runtime

Runtime and Security Model for Hosted Web Applications and Packaged Applications
6 stars 13 forks source link

System Messages .hasPendingMessages name might be incompatible with Mozilla's implementation #107

Open mounirlamouri opened 11 years ago

mounirlamouri commented 11 years ago

Mozilla's implementation is using .mozHasPendingMessage() (without the 's'). If developers end up using this and have prefixed vs un-prefixed paths, it might break when the method get an 's' at the end.

jmajnert commented 11 years ago

Both names seem fine. I guess if you guys already have external devs building apps using this method, then I don't see a reason we shouldn't remove the "s".

anssiko commented 11 years ago

The singular form LGTM. We don't need another s-related issue cf. requestFullscreen() vs. requestFullScreen() :)