910568 - WebTelephony: move telephony DOM classes out of namesapce mozilla::dom::telephony to mozilla::dom

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, defect)

Description

[Vicamo Yang [:vicamo][:vyang]]

From bug 864485 comment 31, let's move public native telephony DOM classes out of namesapce mozilla::dom::telephony to mozilla::dom while still keep private implementation details in it.

Comment 1

[Vicamo Yang [:vicamo][:vyang]]

Attachment: patch

* mozilla::dom - CallEvent.h, CallsList.h, Telephony.h, TelephonyCall.h, TelephonyCallGroup.h

* mozilla::dom::telephony - TelephonyCommon.h, TelephonyFactory.h, TelephonyChild.h, TelephonyParent.h

https://tbpl.mozilla.org/?tree=Try&rev=09dd8a1e5949

Comment 2

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Comment on attachment 802257 [details] [diff] [review]
patch

Review of attachment 802257 [details] [diff] [review]:
-----------------------------------------------------------------

Nice.

::: dom/bindings/Bindings.conf
@@ +166,5 @@
>      'headerFile': 'BluetoothManager.h'
>  },
>  
>  'CallEvent': {
> +    'nativeType': 'mozilla::dom::CallEvent',

The default nativeType for 'Interface' is 'mozilla::dom::Interface', so I don't think you need any of these nativeType annotations anymore.

::: dom/telephony/TelephonyCallGroup.h
@@ +111,5 @@
>  
>    bool CanConference(const TelephonyCall& aCall, TelephonyCall* aSecondCall);
>  };
>  
> +} // namespace mozilla::dom

Just 'namespace dom'

Comment 3

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Also unless Hsin-Yi is actually going to review this please remove her name from the commit message.

Comment 4

[Vicamo Yang [:vicamo][:vyang]]

Attachment: patch : v2

Address previous comment.  Ask for hsinyi's review as well.

Comment 5

[Hsin-Yi Tsai (she/her) [:hsinyi]]

Comment on attachment 802905 [details] [diff] [review]
patch : v2

Review of attachment 802905 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me. But I am not a DOM peer and not able to review this. Remember to remove my name from the bug commit. Thank you.

Comment 6

[Vicamo Yang [:vicamo][:vyang]]

Attachment: patch : v3

update r= only.

Comment 7

[Vicamo Yang [:vicamo][:vyang]]

https://hg.mozilla.org/integration/b2g-inbound/rev/2d38a1453c3d

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/2d38a1453c3d

Comment 9

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

You should probably remove the empty dicts from Bindings.conf.

Comment 10

[Vicamo Yang [:vicamo][:vyang]]

(In reply to :Ms2ger from comment #9)
> You should probably remove the empty dicts from Bindings.conf.

Filed as bug 922531.