888821 - B2G MMS: UA profile configuration based on MNC and MCC.

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, defect, P2)

Description

[Leo]

1. Title: UA profile configuration based on MNC and MCC
2. Precondition: MMS feature enabled on FFOS device
3. Detailed Symptom (ENG.) : When we receive an MMS,there is an URL send to server which contains all the information about the MIME types supported.
In APN,we have set the maximum size for an MMS when sending based on the operator,even at the receiving side we need to have this configurations enabled based on the operator.

Comment 1

[Leo]

The implementation of the UA profile configuration based on MNC and MCC is discussed with Corey in Taiwan workweek

Comment 2

[Gene Lian (I already quit Mozilla)]

Sounds like we can handle this kind of configuration on the Gaia end. Maybe apn.json? I guess.

Comment 3

[Corey Frang [:gnarf]]

So, the only thing that will be different about the UA profile per provider is the max size yeah?

Perhaps we should just have a list of profiles that are based on the max size we already setup in the apn.json?

profile-300k.xml
profile-500k.xml
profile-1024k.xml 

etc?

Is it possible to have this be generated by the server?

uaprofile.xml?operatorDefinedLimit=307200

Comment 4

[Leo]

Hello Corey,

From MMS application point of view, the only difference in UA profile is the Maximum size. Your suggestion on server generating is interesting, however this is not the case. Based on the configuration the client should pick one of these profiles and place it part of the GET request.

Comment 5

[Corey Frang [:gnarf]]

Okay, since the UA profile is sent inside the gecko code, this is something for one of those engineers I think.  We will need a list of the correct URLs.  ctai, gene, is this for one of you?

Comment 6

[Chia-hung Tai [:ctai :ctai_mozilla :cht]]

Because UAProf is device dependent one, I don't suggest use a URLs in here. Those correct URL should be put in customized file. Per talk with Leo guys, my suggestion is below.
We will get the max size of MMS from apn.json. For example, we have wap.UAProf.url.300, wap.UAProf.url.500, wap.UAProf.url.600, wap.UAProf.url.default in b2g.js or user.js. In user.js, it will be a correct url defined by device maker. The Gecko will read the UAProf url by combine the max_size with UAProf. If the max size is 500, Gecko will get the preference "wap.UAProf.url.500". If this preference is not defined, we will use wap.UAProf.url.default.

Comment 7

[Leo]

Attachment: Patch for UAProfile based on MNC and MCC

Hi,
Please verify changes done for UAProfile configuration based on MNC&MCC and provide comments.
Thanks

Comment 8

[Leo]

Attachment: Patch for UAProfile based on MNC and MCC

Hi,
Please check the changes done for UAProfile configuration based on MNC&MCC and provide comments.
Thanks

Comment 9

[Leo]

Comment on attachment 774476 [details]
Patch for UAProfile based on MNC and MCC

Please ignore the attachments

Comment 10

[Chia-hung Tai [:ctai :ctai_mozilla :cht]]

Attachment: b2g18-Patch1.0

Comment 11

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

Comment on attachment 776249 [details] [diff] [review]
b2g18-Patch1.0

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

The only thing we should do in this bug, is expose both "wap.UAProf.url" and "wap.UAProf.tagname" as new Settings entries.  Please do not build "policy" into Gecko.

Comment 12

[Chia-hung Tai [:ctai :ctai_mozilla :cht]]

Attachment: Patch v1.0

Comment 13

[Chia-hung Tai [:ctai :ctai_mozilla :cht]]

Per talk with Vicamo, Steve and LEO guys, we decide to do below method for this bug.
We only expose "wap.UAProf.url" and "wap.UAProf.tagname" as new Settings entries in Gecko part.
We will provide a field "UAProf" in apn.json for different MCC, MNC in Gaia part.
We will left empty by default. Device maker can override those values if necessary.
We need to provide a variable for the default value in Gaia part about the UAProf.url if the values is not presented.

Comment 14

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

Comment on attachment 776911 [details] [diff] [review]
Patch v1.0

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

Please also rewrite the commit summary to something more accurate.

Comment 15

[Chia-hung Tai [:ctai :ctai_mozilla :cht]]

Attachment: Patch v1.1

Comment 16

[Chia-hung Tai [:ctai :ctai_mozilla :cht]]

Attachment: b2g18-Patch v1.1

Comment 17

[Chia-hung Tai [:ctai :ctai_mozilla :cht]]

Comment on attachment 776935 [details] [diff] [review]
b2g18-Patch v1.1

We don't need a b2g18-sepcificed patch.

Comment 18

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

https://hg.mozilla.org/projects/birch/rev/56c849c40fcf

Comment 19

[Chia-hung Tai [:ctai :ctai_mozilla :cht]]

Geko's part is done. Re-assign to Steve Chung. He will take care Gaia's part.

Comment 20

[John Hu [:johnhu][:johu][:醬糊小弟]]

We will use build customization mechanism for device maker to create its owned configuration. The mechanism is based on this page: https://wiki.mozilla.org/B2G/MarketCustomizations. It will be the second app using Runtime Customization which means it changes wap.UAProf.url when mcc/mnc is changed.

The customization file name will be "wapuaprof.json". The file structure is similar to browser, linked with mcc and mnc with zero padding, like:
   {
     "000000": {
       "url": "http://example.url/file.xml"
     },
     "123001": {
       "url": "http://example.url/file.xml"
     }
   }

If "000000" is used, that will be the default value. So, when we can't find the customized record based on mcc/mnc, we use this value as the default value. Otherwise, we use "undefined" as the default value.

After discussing with Chai-hung, the implementation will only support url. When tagname is needed to expose to device maker, we will open another bug to add it.

Comment 21

[John Hu [:johnhu][:johu][:醬糊小弟]]

Attachment: patch for gaia part

1. add wapuaprof.json file to application_data.js
2. add the code to handling WAP UA profile url settings in operator_variant.js
3. add apps/system/resources/wapuaprof.json to ignore list

The implementation is based on the statements of last comment.

Comment 22

[John Hu [:johnhu][:johu][:醬糊小弟]]

I will change the wiki page of https://wiki.mozilla.org/B2G/MarketCustomizations when this patch is r+ and before landing.

Comment 23

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/56c849c40fcf

Comment 24

[Arthur Chen [:arthurcc][inactive after 6/30]]

Comment on attachment 777021 [details]
patch for gaia part

Nice work, John! r=me.

Comment 25

[John Hu [:johnhu][:johu][:醬糊小弟]]

merged to master:
https://github.com/mozilla-b2g/gaia/commit/4ec7c428f6a63a44f888ea6f6ade0385c89ae305

Comment 26

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g18/rev/34cf7d19e76b

Leaving status-b2g18 set to affected until the Gaia patch in comment 25 is uplifted to v1-train.

Comment 27

[John Ford [:jhford] CET/CEST Berlin Time]

There is a merge conflict on v1-train in

apps/system/js/operator_variant/operator_variant.js

Comment 28

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g18_v1_1_0_hd/rev/34cf7d19e76b

John, can you please assist with resolving the v1-train merge conflict in comment 27?

Comment 29

[John Hu [:johnhu][:johu][:醬糊小弟]]

Hi John Ford, 

I had resolved the conflict of operator_variant.js. Please find the branch in my github:

https://github.com/huchengtw-moz/gaia/tree/system/Bug_888821-v1-train

Thanks for help...

John Hu

Comment 30

[John Ford [:jhford] CET/CEST Berlin Time]

v1-train: c376446

Comment 31

[John Ford [:jhford] CET/CEST Berlin Time]

v1.1.0hd: c376446da64aacb6907b26542a2684d54a0162eb
v1.1.0hd: 16db10014db8da6c25b673953264f358d205e206