[FlyWeb] Switch back to using DummyMulticastDNSManager instead of NsdMulticastDNSManager on Android

NEW
Unassigned

Status

()

Core
Networking
P5
normal
2 years ago
3 months ago

People

(Reporter: djvj, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [necko-would-take])

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
Created attachment 8758855 [details] [diff] [review]
switch-back-to-dummy-multicast-dns-manager.patch

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 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+
Whiteboard: [necko-would-take]
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.