1309719 - Remove DOM/telephony and related code.

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Comment 1

[Michelangelo De Simone [Offline] [:mds]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=bafd3fb4f4ae

Comment 2

[Michelangelo De Simone [Offline] [:mds]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=45b45adccfec

Comment 3

[Michelangelo De Simone [Offline] [:mds]]

Attachment: Bug 1309719 - Remove DOM/telephony and related code.

Review commit: https://reviewboard.mozilla.org/r/87430/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/87430/

Comment 4

[Michelangelo De Simone [Offline] [:mds]]

Comment on attachment 8803181 [details]
Bug 1309719 - Remove DOM/telephony and related code.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/87430/diff/1-2/

Comment 5

[(no longer active)]

Comment on attachment 8803181 [details]
Bug 1309719 - Remove DOM/telephony and related code.

https://reviewboard.mozilla.org/r/87430/#review86684

nsITelephonyFoo are accessed from the following places:

* b2g/components/ContentPermissionPrompt.js
* dom/bluetooth
* dom/system/gonk

Removing this without also dealing with those places will break stuff.

r=me but you may not want to land this alone as is.

::: browser/installer/package-manifest.in
(Diff revision 1)
>  @RESPATH@/components/dom_permissionsettings.xpt
>  @RESPATH@/components/dom_sidebar.xpt
>  @RESPATH@/components/dom_mobilemessage.xpt
>  @RESPATH@/components/dom_storage.xpt
>  @RESPATH@/components/dom_stylesheets.xpt
> -@RESPATH@/components/dom_telephony.xpt

There's other stuff you should remove from this file.  Everything in the EXTRA_COMPONENTS variables in the moz.build files you have removed.

Comment 6

[Michelangelo De Simone [Offline] [:mds]]

(In reply to :Ehsan Akhgari (Away Oct 25 - Nov 9) from comment #5)

Hey :Ehsan, thanks for your review.

A couple of remarks:

> * dom/bluetooth

This is gonna go away as per bug 1310020. Also, we're not building that anyway, so nothing is gonna brake (trys are green on all platforms).

> ::: browser/installer/package-manifest.in
> (Diff revision 1)
> >  @RESPATH@/components/dom_permissionsettings.xpt
> >  @RESPATH@/components/dom_sidebar.xpt
> >  @RESPATH@/components/dom_mobilemessage.xpt
> >  @RESPATH@/components/dom_storage.xpt
> >  @RESPATH@/components/dom_stylesheets.xpt
> > -@RESPATH@/components/dom_telephony.xpt
> 
> There's other stuff you should remove from this file.  Everything in the
> EXTRA_COMPONENTS variables in the moz.build files you have removed.

AFAICT all the declared EXTRA_COMPONENTS are "compressed" in dom_telephony.xpt alone: packaging works as expected on both desktop and mobile. Would you please hint me to which one(s) I've overlooked, please?

Comment 7

[(no longer active)]

(In reply to Michelangelo De Simone [:mds] from comment #6)
> (In reply to :Ehsan Akhgari (Away Oct 25 - Nov 9) from comment #5)
> 
> Hey :Ehsan, thanks for your review.
> 
> A couple of remarks:
> 
> > * dom/bluetooth
> 
> This is gonna go away as per bug 1310020. Also, we're not building that
> anyway, so nothing is gonna brake (trys are green on all platforms).

Cool!

> > ::: browser/installer/package-manifest.in
> > (Diff revision 1)
> > >  @RESPATH@/components/dom_permissionsettings.xpt
> > >  @RESPATH@/components/dom_sidebar.xpt
> > >  @RESPATH@/components/dom_mobilemessage.xpt
> > >  @RESPATH@/components/dom_storage.xpt
> > >  @RESPATH@/components/dom_stylesheets.xpt
> > > -@RESPATH@/components/dom_telephony.xpt
> > 
> > There's other stuff you should remove from this file.  Everything in the
> > EXTRA_COMPONENTS variables in the moz.build files you have removed.
> 
> AFAICT all the declared EXTRA_COMPONENTS are "compressed" in
> dom_telephony.xpt alone: packaging works as expected on both desktop and
> mobile. Would you please hint me to which one(s) I've overlooked, please?

No, you're thinking of XPIDL_SOURCES.  The xpt files are compiled interface information that we get from processing XPIDL .idl files (the other output of that processing is the corresponding generated .h file)

If you look at EXTRA_COMPONENTS in dom/telephony/moz.build, you'll see:

 63     EXTRA_COMPONENTS += [
 64         'gonk/TelephonyAudioService.js',
 65         'gonk/TelephonyAudioService.manifest',
 66         'gonk/USSDReceivedWrapper.js',
 67         'gonk/USSDReceivedWrapper.manifest',
 68     ]
 69     if not CONFIG['DISABLE_MOZ_RIL_GEOLOC']:
 70         EXTRA_COMPONENTS += [
 71             'gonk/TelephonyService.js',
 72             'gonk/TelephonyService.manifest',
 73         ]

All of these files appear in b2g's package manifest in addition to dom_telephony.xpt and they need to all be removed.

Comment 8

[Michelangelo De Simone [Offline] [:mds]]

Comment on attachment 8803181 [details]
Bug 1309719 - Remove DOM/telephony and related code.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/87430/diff/2-3/

Comment 9

[Michelangelo De Simone [Offline] [:mds]]

Comment on attachment 8803181 [details]
Bug 1309719 - Remove DOM/telephony and related code.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/87430/diff/3-4/

Comment 10

[Johnny Stenback (:jst)]

I think given the complex nature of the intertwined patches we'll be going through in order to follow up on our plan to remove the B2G code from mozilla-central, it's ok for things to end up in a broken state between patch landings. We should of course where it makes sense keep that intermittent breakage to a minimum and cluster patches as reasonable.

IOW, go ahead and land this and try to follow up with the continuation of removals of the broken code.

Comment 11

[Michelangelo De Simone [Offline] [:mds]]

Comment on attachment 8803181 [details]
Bug 1309719 - Remove DOM/telephony and related code.

https://reviewboard.mozilla.org/r/87430/#review86684

> There's other stuff you should remove from this file.  Everything in the EXTRA_COMPONENTS variables in the moz.build files you have removed.

This file was correct:), you meant the b2g one... I've removed the xpt's from that too. Thanks.

Comment 12

[Michelangelo De Simone [Offline] [:mds]]

Comment on attachment 8803181 [details]
Bug 1309719 - Remove DOM/telephony and related code.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/87430/diff/4-5/

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a7e827f90ba9
Remove DOM/telephony and related code. r=ehsan+251051

Comment 14

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/a7e827f90ba9

Comment 15

[Chris Mills [:cmills]]

I've updated the docs to account for these changes.

All relevant pages have been archived:

https://developer.mozilla.org/en-US/docs/Archive/Firefox_OS/API
https://developer.mozilla.org/en-US/docs/Archive/Firefox_OS/Web_Telephony_API

I've also added a note to the Fx 52 release notes:

https://developer.mozilla.org/en-US/Firefox/Releases/52#Others