Closed
Bug 1143984
Opened 9 years ago
Closed 9 years ago
B2G NetworkManager: 'active' should only set for default connections, e.g. wifi or default data
Categories
(Firefox OS Graveyard :: RIL, defect)
Tracking
(tracking-b2g:backlog, firefox41 fixed)
People
(Reporter: jessica, Assigned: jessica)
References
Details
Attachments
(1 file, 2 obsolete files)
2.76 KB,
patch
|
edgar
:
review+
|
Details | Diff | Splinter Review |
Currently, NetworkManager's |active| may be set to a secondary data connection, e.g. mms. However, we think |active| should only be set for default connections, e.g. wifi or default data.
Assignee | ||
Comment 1•9 years ago
|
||
Assignee: nobody → jjong
Assignee | ||
Comment 2•9 years ago
|
||
Comment on attachment 8579270 [details] [diff] [review] patch, v1. I think this is ready. Edgar, may I have your review? Thanks.
Attachment #8579270 -
Attachment description: WIP. → patch, v1.
Attachment #8579270 -
Flags: review?(echen)
Assignee | ||
Comment 3•9 years ago
|
||
Comment on attachment 8579270 [details] [diff] [review] patch, v1. Cancelling review since we need to fix dns default interface issue, see Bug 992772.
Attachment #8579270 -
Flags: review?(echen)
Assignee | ||
Comment 4•9 years ago
|
||
Fix dns default interface issue, not tested yet.
Attachment #8579270 -
Attachment is obsolete: true
Assignee | ||
Comment 5•9 years ago
|
||
Hmm, I'm still finding a way to test SDK_VERSION 18, since base image v123 (JB) seems not to work with master. It makes me wonder if we need that ugly fix for SDK_VERSION 18, since it doesn't work current master and it is no longer supported on Flame: "Note: Firefox OS images v180 and above are based on Android KK (Kitkat, 4.4); JB (Jellybean, 4.1–4.3) builds have now been discontinued and are no longer supported, so don't use anything older than v180." https://developer.mozilla.org/en-US/Firefox_OS/Phone_guide/Flame/Updating_your_Flame
Assignee | ||
Comment 6•9 years ago
|
||
We'll wait for bug 992772 to be landed first.
Comment 8•9 years ago
|
||
(In reply to Jessica Jong [:jjong] [:jessica] from comment #5) > Hmm, I'm still finding a way to test SDK_VERSION 18, since base image v123 > (JB) seems not to work with master. It makes me wonder if we need that ugly > fix for SDK_VERSION 18, since it doesn't work current master and it is no > longer supported on Flame: > > "Note: Firefox OS images v180 and above are based on Android KK (Kitkat, > 4.4); JB (Jellybean, 4.1–4.3) builds have now been discontinued and are no > longer supported, so don't use anything older than v180." > > https://developer.mozilla.org/en-US/Firefox_OS/Phone_guide/Flame/ > Updating_your_Flame I just found OPEN_C is still using jellybean. Please see https://bugzilla.mozilla.org/show_bug.cgi?id=1114935#c150. Looks like we still need to take jellybean into consideration. :(
Assignee | ||
Comment 9•9 years ago
|
||
(In reply to Edgar Chen [:edgar][:echen] (OOO 3/21~4/6) from comment #8) > (In reply to Jessica Jong [:jjong] [:jessica] from comment #5) > > Hmm, I'm still finding a way to test SDK_VERSION 18, since base image v123 > > (JB) seems not to work with master. It makes me wonder if we need that ugly > > fix for SDK_VERSION 18, since it doesn't work current master and it is no > > longer supported on Flame: > > > > "Note: Firefox OS images v180 and above are based on Android KK (Kitkat, > > 4.4); JB (Jellybean, 4.1–4.3) builds have now been discontinued and are no > > longer supported, so don't use anything older than v180." > > > > https://developer.mozilla.org/en-US/Firefox_OS/Phone_guide/Flame/ > > Updating_your_Flame > > I just found OPEN_C is still using jellybean. Please see > https://bugzilla.mozilla.org/show_bug.cgi?id=1114935#c150. > Looks like we still need to take jellybean into consideration. :( Alright, I will let Henry know. :)
Assignee | ||
Comment 10•9 years ago
|
||
Attachment #8579972 -
Attachment is obsolete: true
Assignee | ||
Comment 11•9 years ago
|
||
Comment on attachment 8605703 [details] [diff] [review] patch, v1. Edgar, may I have you review on this, now that bug 992772 has landed? Thanks!
Attachment #8605703 -
Flags: review?(echen)
Comment 12•9 years ago
|
||
Comment on attachment 8605703 [details] [diff] [review] patch, v1. Review of attachment 8605703 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me, thank you.
Attachment #8605703 -
Flags: review?(echen) → review+
Assignee | ||
Comment 13•9 years ago
|
||
try looks good: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2f6f564d2713
You need to log in
before you can comment on or make changes to this bug.
Description
•