1131590 - [Shinano] Unknown network type detected: support DC-HSPA

Status: RESOLVED FIXED

Description

[Alexandre LISSY :gerard-majax]

My SIM (Orange F) is registered on a network that has DC support. Device also support it. In Gecko, however, we do not handle it, resulting in a "no data" icon being disabled when we are connected in DC mode.

Comment 1

[Alexandre LISSY :gerard-majax]

Created attachment 8562088 [details] [diff] [review]
Support DualCarrier r=echen

Comment 2

[Alexandre LISSY :gerard-majax]

Hsin-Yi, can you help me on this ? Should I redirect review to you ?

Comment 3

[Edgar Chen [:edgar]]

Comment on attachment 8562088 [details] [diff] [review]
Support DualCarrier r=echen

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

These two type isn't defined in AOSP standard, I was trying to figure out where is value from by checking caf's git source [1].
But I just realized that maybe it is not from caf and could be an phone manufacturer's specific type. 
Since it looks like we need these types for `Shinano`, I am ok with this changes.

[1] https://www.codeaurora.org/cgit/quic/la/platform/hardware/ril/

r=me with below comments addressed.
Thank you, Alexandre.

::: dom/system/gonk/ril_worker.js
@@ +4372,3 @@
>          return true;
> +      default:
> +        this.context.debug("Unsupported radioTech: " + radioTech);

Since this function is to check if the |radioTech| is GSM family, we don't have to print this kinds of warning, but just return true or false.

@@ +6345,4 @@
>          cellId.wcdmaPsc = this.parseInt(cid, -1, 16);
>          break;
> +      default:
> +        this.context.debug("Unsupported radioTech: " + radioTech);

Ditto.
We already filter out the unsupported radio tech by checking |_isGsmTechGroup| [2], don't have to print any warning here.

[2] https://dxr.mozilla.org/mozilla-central/source/dom/system/gonk/ril_worker.js#6299-6303

Comment 4

[Alexandre LISSY :gerard-majax]

Created attachment 8564039 [details] [diff] [review]
Support DualCarrier r=echen

Comment 5

[Alexandre LISSY :gerard-majax]

Comment on attachment 8564039 [details] [diff] [review]
Support DualCarrier r=echen

Carrying r+

Edgar, do you mind pushing this for me ? I'm gonna be out of network for a little while and I'd like this to land soon :). Thanks!

Comment 6

[Edgar Chen [:edgar]]

b2g-inbound is closed now. :(
I will try pushing it later.

Comment 7

[Edgar Chen [:edgar]]

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

Comment 8

[Carsten Book [:Tomcat]]

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