Closed Bug 1507861 Opened 6 years ago Closed 5 years ago

Move dispatch-to-STS logic into MediaTransportHandler

Tracking

()

Status:

RESOLVED WONTFIX

People

(Reporter: bwc, Assigned: bwc)

References

(Blocks 1 open bug)

Details

Attachments

(5 files, 2 obsolete files)

Bug 1507861: (WIP) Move STS thread dispatches into MediaTransportHandler. 6 years ago Byron Campen [:bwc] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1507861: (WIP) Move STS thread dispatches into MediaTransportHandler. 6 years ago Byron Campen [:bwc] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1507861 - Part 1: Move STS thread dispatches into MediaTransportHandler. 6 years ago Byron Campen [:bwc] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1507861 - Part 2: Stop using STS in conduit code. 6 years ago Byron Campen [:bwc] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1507861 - Part 4: Remove unnecessary locking in VideoConduit, along with some minor fixes/simplifications. 6 years ago Byron Campen [:bwc] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1507861 - Part 5: Various fixes in AudioConduit, and remove unnecessary locking. 6 years ago Byron Campen [:bwc] 47 bytes, text/x-phabricator-request		Details \| Review
Bug 1507861 - Part 3: Remove STS from DataChannel.h/cpp 6 years ago Byron Campen [:bwc] 47 bytes, text/x-phabricator-request		Details \| Review

Byron Campen [:bwc]

Assignee

Description

•

6 years ago

We want to present the same API surface regardless of whether the underlying implementation uses STS or IPC.

Byron Campen [:bwc]

Assignee

Updated

•

6 years ago

Assignee: nobody → docfaraday

Byron Campen [:bwc]

Assignee

Comment 1

•

6 years ago

Attached file Bug 1507861: (WIP) Move STS thread dispatches into MediaTransportHandler. (obsolete) — Details

Byron Campen [:bwc]

Assignee

Comment 2

•

6 years ago

It looks like this will afford an opportunity to really clean up the threading model for the signaling side as well.

Nico Grunbaum [:ng, @chew:mozilla.org]

Updated

•

6 years ago

Rank: 20

Priority: -- → P2

Nico Grunbaum [:ng, @chew:mozilla.org]

Updated

•

6 years ago

Rank: 20 → 19

Byron Campen [:bwc]

Assignee

Comment 3

•

6 years ago

Attached file Bug 1507861: (WIP) Move STS thread dispatches into MediaTransportHandler. (obsolete) — Details

Byron Campen [:bwc]

Assignee

Comment 4

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=2a8c9fc5ce15a8709edeeca1a46b1e7bdfaaec2b

Byron Campen [:bwc]

Assignee

Comment 5

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=130747d91b2848ef5a4f25187676959d596248ce

Byron Campen [:bwc]

Assignee

Updated

•

6 years ago

Attachment #9025792 - Attachment is obsolete: true

Byron Campen [:bwc]

Assignee

Comment 8

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=ed47de68117e91cbc7dd0ef413e0e13fa6d75e99

Byron Campen [:bwc]

Assignee

Comment 9

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=8986eae69e3e76d443b3f7a1d8234e7c12197dd6

Byron Campen [:bwc]

Assignee

Comment 10

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e49feb498ad4f3b6cf9dc59c1946a00551f5f5a3

Byron Campen [:bwc]

Assignee

Comment 11

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=1cc2a85370a93e7c78f5ca787cf2813d0e37364e

Byron Campen [:bwc]

Assignee

Comment 12

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=1ba5d82ed26d7d86984a0d8267446634c56958b7

Byron Campen [:bwc]

Assignee

Comment 13

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=cabeef0e194d6cedd5499666e0eb6b5414da8c90

Byron Campen [:bwc]

Assignee

Comment 14

•

6 years ago

Attached file Bug 1507861 - Part 1: Move STS thread dispatches into MediaTransportHandler. — Details

Byron Campen [:bwc]

Assignee

Comment 15

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e12d93f87f3ff0e0b057ff1e60138a1cfa66d289

Byron Campen [:bwc]

Assignee

Comment 16

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=bc5a85ae8dca611fd51e83b181790e20ac7de421

Honza Bambas (:mayhemer)

Updated

•

6 years ago

Blocks: socket-proc-webrtc
No longer blocks: socket-proc

Byron Campen [:bwc]

Assignee

Comment 17

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=d763960fb84c0d9e6e6832d95b3e9831291ff63f

Byron Campen [:bwc]

Assignee

Comment 18

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a6a67ad07d230da330081bd9bb4be2f1dd5a817e

Byron Campen [:bwc]

Assignee

Comment 19

•

6 years ago

Attached file Bug 1507861 - Part 2: Stop using STS in conduit code. — Details

Byron Campen [:bwc]

Assignee

Comment 20

•

6 years ago

Attached file Bug 1507861 - Part 4: Remove unnecessary locking in VideoConduit, along with some minor fixes/simplifications. — Details

Byron Campen [:bwc]

Assignee

Comment 21

•

6 years ago

Attached file Bug 1507861 - Part 5: Various fixes in AudioConduit, and remove unnecessary locking. — Details

Phabricator Automation

Updated

•

6 years ago

Attachment #9026177 - Attachment is obsolete: true

Byron Campen [:bwc]

Assignee

Comment 22

•

6 years ago

Attached file Bug 1507861 - Part 3: Remove STS from DataChannel.h/cpp — Details

Byron Campen [:bwc]

Assignee

Comment 23

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9cfd2d9f0bad4eb8120958b300f700252c821cb9

Byron Campen [:bwc]

Assignee

Comment 24

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=51b1d28e1e1edd190f8e229f3911f17a7360ca99

Byron Campen [:bwc]

Assignee

Comment 25

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=100c38db0d9470a41a7afc0784dd437511bc3ec7

Byron Campen [:bwc]

Assignee

Comment 26

•

6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=1898d5be73bf34e1e0fbae7c494edda7c696462e

Nico Grunbaum [:ng, @chew:mozilla.org]

Updated

•

6 years ago

Blocks: 1499796

BugBot [:suhaib / :marco/ :calixte]

Comment 27

•

5 years ago

There are some r+ patches which didn't land and no activity in this bug for 2 weeks.
:bwc, could you have a look please?

Flags: needinfo?(docfaraday)

Byron Campen [:bwc]

Assignee

Comment 28

•

5 years ago

Unfortunately, this stuff is going to have to be re-thought.

Status: NEW → RESOLVED

Closed: 5 years ago

Flags: needinfo?(docfaraday)

Resolution: --- → WONTFIX

You need to log in before you can comment on or make changes to this bug.