1005624 - In setting->device information->more information->developer, If the checkbox of Enable hardware compo... is checked,setting activity will exit occasionally

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Settings, defect)

Description

[Shiwei Zhang]

+++ This bug was initially created as a clone of Bug #1002327 +++

Steps to reproduce:

I'm from Spreadtrum corporation and we are now developing engmode on version v1.3.

The window.navigator.mozMobileConnection.sendMMI('*#06#') fuction works well on one sim card version. We could get the imei number from the dialer interface or the setting app.

But when we try to use the function to get the imei on two sim card slots device,it returns only one imei.


Actual results:

Only one imei returned

Expected results:

Two imei returned, because there are two sim card slots.

Pls give me a hand .

Tks a lot.

Comment 1

[Shiwei Zhang]

Please ignore the above and pay attention to this

In setting->device information->more information->developer, If the checkbox of Enable hardware compo... is checked,setting  activity will exit occasionally

Comment 2

[Shiwei Zhang]

I'm from Spreadtrum corporation and we found crash will happen occasionally when the checkbox of Enable hardware compo... is checked.

What we are suggesting is that the checkbox of Enable hardware compo... can be hided or removed.

If convenient，please see further information at the link
http://bugzilla.spreadtrum.com/bugzilla/show_bug.cgi?id=308147

Thanks a lot.

Comment 3

[James Zhang (Spreadtrum)]

Please attach your patch here.

Comment 4

[Shiwei Zhang]

Attachment: settings_about_more_info_developer_patch

Attachment is a patch dealing with the issue

Comment 5

[[:fabrice] Fabrice Desré]

Comment on attachment 8417143 [details] [diff] [review]
settings_about_more_info_developer_patch

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

Please land on 1.3t only, with the following commit message:

Bug 1005624: Remove the hwc settings option on tarako r=fabrice

thanks!

Comment 6

[Joe Cheng [:jcheng] (please needinfo)]

triage: let's take this change for tarako only. 1.3T+

Comment 7

[Keven Kuo [:kkuo]]

Check in needed.

--
Keven

Comment 8

[James Zhang (Spreadtrum)]

Please land it.

Comment 9

[[:fabrice] Fabrice Desré]

Pushed as 3f12b99379c3ffa98c847b8fda1bbf8d6bbd8c39 on 1.3t.

Guys, I'm happy to help but I would really appreciate that people get patches in a landable state as I asked in comment 5.

Comment 10

[yang.zhao]

(In reply to James Zhang from comment #8)
> Please land it.

See comment 9,Fabrice already landed.
commit:3f12b99379c3ffa98c847b8fda1bbf8d6bbd8c39