ChannelMergerNode supports active processing #17273
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When a ChannelMergerNode is not actively processing its inputs, the
number of channels in its output is one, instead of the number of
inputs.
Just need to disable outputs when the node is created. Connections to
the node will adjust the number of channels appripriately, and when
inputs are no longer actively processing, the number of output
channels becomes one, as required.
Bug: 925046
Test: the-channelmergernode-interface/active-processing.https.html
Change-Id: I08e5fd2fe9c99b75af177eb85212162ecf39efd4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645478
Reviewed-by: Hongchan Choi <hongchan@chromium.org>
Commit-Queue: Raymond Toy <rtoy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#668159}