Closed
Bug 1245963
Opened 8 years ago
Closed 8 years ago
Integrate Kannan and Jonas FlyWeb code
Categories
(Core :: Networking, defect)
Core
Networking
Tracking
()
RESOLVED
FIXED
People
(Reporter: djvj, Unassigned)
References
Details
Attachments
(3 files, 2 obsolete files)
50.05 KB,
patch
|
Details | Diff | Splinter Review | |
15.67 KB,
patch
|
Details | Diff | Splinter Review | |
19.86 KB,
patch
|
Details | Diff | Splinter Review |
Jonas has been working on the server implementation, and I've been working on the discovery/naming/transport layer implementation. This has led to a fragmentation of similar objects which should be unified into one set of implementation objects.
Reporter | ||
Comment 1•8 years ago
|
||
This patch moves most of the FlyWebDiscoveryManager implementation into FlyWebService. FlyWebDiscoveryManager becomes a thin forwarder object around FlyWebService.
Reporter | ||
Comment 2•8 years ago
|
||
Refactors the publish server implementation to separate the frontend JS object from the (potentially many, eventually) backend implementation objects. Not tested. Probably has bugs, probably will crash. Sicking: can you walk me through how you smoke-test your flyweb:// channel/protocol implementation so far? Do you have some test pages somewhere? I need to go through this to test/debug the code.
Reporter | ||
Comment 3•8 years ago
|
||
Forgot to ni, so re-posting comment: Sicking: can you walk me through how you smoke-test your flyweb:// channel/protocol implementation so far? Do you have some test pages somewhere? I need to go through this to test/debug the code.
Flags: needinfo?(jonas)
Flags: needinfo?(jonas)
Reporter | ||
Comment 6•8 years ago
|
||
This makes it so that servers published using |navigator.publishServer| are advertised via the mdns backend.
Reporter | ||
Comment 7•8 years ago
|
||
Pushed patches: http://hg.mozilla.org/projects/larch/rev/ab9bd2ff3c13 http://hg.mozilla.org/projects/larch/rev/3917c13c8b11 http://hg.mozilla.org/projects/larch/rev/c879d7be4fdf
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•