Closed Bug 1020167 Opened 6 years ago Closed 6 years ago
[Single Variant] Add search engine configuration based on MCC/MNC
Add single variant configuration to overwrite the search engine settings if provided in an operator customisation, based on MCC/MNC combination: - search.urlTemplate - search.suggestionsUrlTemplate - search.iconUrl - search.providers
Patch updated, now it sets the property 'search.providers' with the list of search providers for the mcc/mnc of the SIM.
Comment on attachment 8434010 [details] Patch Thanks! I had a quick look over this and it looks good, I just left a couple of comments on Github. Will let Yuren do the full review.
Attachment #8434010 - Flags: feedback?(bfrancis) → feedback+
Comment on attachment 8434010 [details] Patch Albert and Ben the only one issue is property of configuration "iconUrl", since we used iconPath on bug 1009633 and generated iconUri field in build_stage, we should use same property name on bug 1020167, bug 1009633 and bug 1009353. I propose using iconPath for those three bugs and generate iconUri if needed, does it make sense?
For me is ok, I change it. Ben, it doesn't mean any change in your main code, just in the build process.
Comment on attachment 8434010 [details] Patch Changed iconUrl by iconPath in json. iconUrl is created at build time.
Comment on attachment 8434010 [details] Patch thanks!
Attachment #8434010 - Flags: review?(yurenju.mozilla) → review+
and please update MDN article for this customization. https://developer.mozilla.org/en-US/Firefox_OS/Developing_Firefox_OS/Market_customizations_guide
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Verified on master ( see comment in bug 1031524 ) blocked by bug 1009443 on testing on flame. Verified fixed on 2.0 buri Gaia d9b225fd156b16adabcf459275b52674dde517cc Gecko https://hg.mozilla.org/releases/mozilla-aurora/rev/0b5f757da75a BuildID 20140709000201 Version 32.0a2 ro.build.version.incremental=324 ro.build.date=Thu Dec 19 14:04:55 CST 2013
Tester's Note : Albert had set me straight. bug 1031164 is what I am really blocked on for flame. This does not handle DSDS cases. Bug 927709 will handle DSDS cases.
You need to log in before you can comment on or make changes to this bug.