Closed Bug 1165844 Opened 9 years ago Closed 9 years ago

Add stopDhcp function to nsINetworkService

Categories

(Firefox OS Graveyard :: General, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(firefox41 fixed)

RESOLVED FIXED
2.2 S13 (29may)
Tracking Status
firefox41 --- fixed

People

(Reporter: hchang, Assigned: hchang)

References

Details

Attachments

(1 file, 1 obsolete file)

Since Bug 1165776 will run dhcp_request library call in a separate thread, 'stopDhcp' becomes able to interrupt pending dhcp request. This bug is to add
'stopDhcp' function to nsINetworkService.
Assignee: nobody → hchang
Attached patch Bug1165844.patch (obsolete) — Splinter Review
Blocks: 1000040
Attachment #8606955 - Flags: review?(vchang)
Comment on attachment 8606955 [details] [diff] [review]
Bug1165844.patch

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

Looks good, thank you. 
Please open a bug to refactor wifinetutil.jsm
Attachment #8606955 - Flags: review?(vchang) → review+
Blocks: 1166212
Keywords: checkin-needed
(In reply to Vincent Chang[:vchang] from comment #2)
> Comment on attachment 8606955 [details] [diff] [review]
> Bug1165844.patch
> 
> Review of attachment 8606955 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Looks good, thank you. 
> Please open a bug to refactor wifinetutil.jsm

Thanks for the review. I've already filed a bug to track the refactoring.
Thanks!
Keywords: checkin-needed
Attachment #8606955 - Attachment is obsolete: true
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/679cbdf6d143
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S13 (29may)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: