Open
Bug 1277350
Opened 8 years ago
Updated 2 years ago
[FlyWeb] Switch back to using DummyMulticastDNSManager instead of NsdMulticastDNSManager on Android
Categories
(Core :: Networking, defect, P5)
Core
Networking
Tracking
()
NEW
People
(Reporter: djvj, Unassigned)
Details
(Whiteboard: [necko-would-take])
Attachments
(1 file)
|
1.19 KB,
patch
|
justindarc
:
review+
|
Details | Diff | Splinter Review |
The original impl of MulticastDNSManager on java used to be DummyMulticastDNSManager, but we changed it a long time ago to use NsdMulticastDNSManager because we needed it to be non-dummy. We have since bypassed the entire android implementation with our JS implementation, so we can eliminate this change.
Attachment #8758855 -
Flags: review?(jdarcangelo)
|
||
Comment 1•8 years ago
|
||
Comment on attachment 8758855 [details] [diff] [review] switch-back-to-dummy-multicast-dns-manager.patch Review of attachment 8758855 [details] [diff] [review]: ----------------------------------------------------------------- I was already including this same change in the patch for Fennec I'm already working on. But, this is fine to land on its own too.
Attachment #8758855 -
Flags: review?(jdarcangelo) → review+
|
||
Updated•8 years ago
|
Whiteboard: [necko-would-take]
|
||
Comment 2•7 years ago
|
||
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: -- → P5
|
||
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•