We decided about a year ago that the permissions required to ship WebSMS were too scary. They turned off many users without offering any added benefit for them since no existing web apps used the API. I'm hoping this set up can be a good compromise. When the API gets exercised for the first time, Firefox asks the user to install a helper service that will request the additional permissions. This WIP patch does just enough to send an SMS. It has a menu item to trigger the APK installation (CC'ing Ian to figure out what UX we want around that) and comments out the PermissionManager calls for testing.

Comment on attachment 735461 [details] [diff] [review] WIP patch Review of attachment 735461 [details] [diff] [review]: ----------------------------------------------------------------- I'm not really sure what my feedback is for exactly. Generally speaking, it looks good. The hack is quite nice ;) The only two comments I have are listed below. ::: mobile/android/base/GeckoSmsProtocol.java @@ +2,5 @@ > + > +public class GeckoSmsProtocol { > + public static final int VERSION = 0; > + public static final int ACTION_QUERY_VERSION = 1; > + public static final int ACTION_SEND = 2; I'm not sure I understand. Does that mean that Firefox Android will only ask the service to send stuff? ::: mobile/android/confvars.sh @@ +51,4 @@ > > MOZ_FENNEC=1 > > +MOZ_WEBSMS_BACKEND=1 I think we could actually simply remove MOZ_WEBSMS_BACKEND in that case. It has been created only for Firefox Android.

(In reply to Mounir Lamouri (:mounir) from comment #1) > Comment on attachment 735461 [details] [diff] [review] > WIP patch > > Review of attachment 735461 [details] [diff] [review]: > ----------------------------------------------------------------- > > I'm not really sure what my feedback is for exactly. Generally speaking, it > looks good. The hack is quite nice ;) > > The only two comments I have are listed below. > > ::: mobile/android/base/GeckoSmsProtocol.java > @@ +2,5 @@ > > + > > +public class GeckoSmsProtocol { > > + public static final int VERSION = 0; > > + public static final int ACTION_QUERY_VERSION = 1; > > + public static final int ACTION_SEND = 2; > > I'm not sure I understand. Does that mean that Firefox Android will only ask > the service to send stuff? No, I just have only plumbed up the IPC for send() so far. That list will grow as I flesh out the rest of it.

filter on [mass-p5]