1033183 - [Wifi] Server certificate items are duplicated when disabling and re-enabling wifi

Status: VERIFIED FIXED

Description

[Gerry Chang [:gchang]]

### Steps:
1. Import 2 server certificates from Settings > Wi-Fi > Manage certificates
2. Close Settings app
3. Open Settings > Wi-Fi page
4. Select a EAP WLAN, EAP method is TTLS
5. Tap "Server certficate"
6. There are 2 certificates listed here, which is expected
7. Click "<" to go back to previous page and disable Wi-Fi
8. Enable Wi-Fi again
9. Select a EAP WLAN again
10. Tap "Server certficate" again
11. One of the certificates is duplicated

### Expected:
There should be no certificates duplicated here.

### Actual:
One certificate is duplicated.

### Reproduce rate
always

### Version:
Device    flame
Gaia      43226cf5c3ad19728a88b3786595670b6d60e5c6
Gecko     https://hg.mozilla.org/releases/mozilla-aurora/rev/d0275ba5f438
BuildID   20140701160202
Version   32.0a2

Comment 1

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Ian, could you help?

Comment 2

[[e-mail], [e-mail]]

Yes, take over the issue.

Comment 3

[[e-mail], [e-mail]]

Created attachment 8451509 [details] [review]
pull request 21438

Arthur, could you please help to review my pull request? Thanks.

Comment 4

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

Comment on attachment 8451509 [details] [review]
pull request 21438

r=me, thank you.

Comment 5

[[e-mail], [e-mail]]

Just rebase the patch for verification test again.

Comment 6

[[e-mail], [e-mail]]

Since the pr is landed, we can close the issue now.

Gaia/master:  e970c7e69c213fd896383b0dc7b85a1b7b1d7dc8

Comment 7

[[e-mail], [e-mail]]

Gaia/v2.0:  35a9b715e7348ec738ff6c8a59f50190390a06f2

Comment 8

[Ryan VanderMeulen [:RyanVM]]

FYI, checkin-needed means "please help me land this" which clearly wasn't the case here.

Comment 9

[Gerry Chang [:gchang]]

Verified @
Gaia      ca022f811bcbbda0f89086094a9e92bb220fea18
Gecko     https://hg.mozilla.org/releases/mozilla-aurora/rev/376889ab0e02
BuildID   20140713160202
Version   32.0a2

Comment 10

[Shally Li（leave from Mozilla）]

Hi Reporter,
    Could you provide the certificates for me to verify this bug on flame v2.1?

Thank you!

Comment 11

[Gerry Chang [:gchang]]

Created attachment 8533479 [details]
A server certificate.

Comment 12

[Gerry Chang [:gchang]]

Created attachment 8533480 [details]
Another server certificate.

Comment 13

[Shally Li（leave from Mozilla）]

Per Comment 9,set Flame v2.0 as "verified".

-------------------------------------------------
This bug has been failed verified on latest Flame v2.1 (Rate:5/5,see "verified_v2.1.mp4"). 
I will submit a following up bug tomorrow.

Repro STR:
1. Copy the two server certificate to device's storage.
2. Open "Settings"->"Manage certificates"->"Import certificate",then import the certificates.
3. Select a EAP WLAN, EAP method is TTLS.
4. Tap "Server certficate".
5. There are 2 certificates listed here.
6. Tap "Ok" and tap "<" to go back to Wi-fi list.
8. Select the EAP WLAN again.
9. Tap "Server certficate" again.
**One certificate is duplicated. --KO

Comment 14

[Shally Li（leave from Mozilla）]

Created attachment 8579243 [details]
verified_v2.1.mp4

Comment 15

[Shally Li（leave from Mozilla）]

The following up bug is bug 1145031.

Comment 16

[Shally Li（leave from Mozilla）]

Hi Mike,
   
    This bug has been failed verified on latest Flame v2.1 in Comment 13,and I had filed a following up bug. Could you help with the following up bug (1145031)?

Thank you very much.

Comment 17

[Mike Lien[:mlien]]

Already ni developer from the following bug, keep tracking