The default bug view has changed. See this FAQ.

Integrate Kannan and Jonas FlyWeb code

RESOLVED FIXED

Status

()

Core
Networking
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: djvj, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments, 2 obsolete attachments)

(Reporter)

Description

a year ago
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

a year ago
Created attachment 8715987 [details] [diff] [review]
1-move-flyweb-discovery-manager-into-flyweb-service.patch

This patch moves most of the FlyWebDiscoveryManager implementation into FlyWebService.  FlyWebDiscoveryManager becomes a thin forwarder object around FlyWebService.
(Reporter)

Comment 2

a year ago
Created attachment 8716467 [details] [diff] [review]
2-refactor-publish-server-implementation.patch

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

a year 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 4

a year ago
Created attachment 8718554 [details] [diff] [review]
1-move-flyweb-discovery-manager-into-flyweb-service.patch

Updated patch.
Attachment #8715987 - Attachment is obsolete: true
(Reporter)

Comment 5

a year ago
Created attachment 8718555 [details] [diff] [review]
2-refactor-publish-server-implementation.patch

Updated patch.
Attachment #8716467 - Attachment is obsolete: true
(Reporter)

Comment 6

a year ago
Created attachment 8718556 [details] [diff] [review]
3-add-mdns-advertising.patch

This makes it so that servers published using |navigator.publishServer| are advertised via the mdns backend.
(Reporter)

Comment 7

a year 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
Last Resolved: a year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.