New issue
Advanced search Search tips

Issue 904764 link

Starred by 3 users

Issue metadata

Status: Assigned
Owner:
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature



Sign in to add a comment

Tracking bug for WebRTC max jitter buffer size origin trial

Project Member Reported by kuddai@google.com, Nov 13

Issue description

We plan to run origin trial which allows us to set maximum audio jitter buffer size from javascript layer.

Maximum audio jitter buffer size controls how many sample we can accumulate before we flush the audio buffer.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 16

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/537539576bdbbdfcd46d93941b1a5ca9b57481fe

commit 537539576bdbbdfcd46d93941b1a5ca9b57481fe
Author: Ruslan Burakov <kuddai@google.com>
Date: Fri Nov 16 13:31:39 2018

make jitter_buffer_max_size available in blink layer under origin trial

link to the experiment thread
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/hE2B1iItPDk

Bug: chromium:904764
Change-Id: Ic89e7915a21c60868e0a25b5a9ba160956f86c69
Reviewed-on: https://chromium-review.googlesource.com/c/1332194
Commit-Queue: Ruslan Burakov <kuddai@google.com>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608752}
[add] https://crrev.com/537539576bdbbdfcd46d93941b1a5ca9b57481fe/third_party/WebKit/LayoutTests/http/tests/origin_trials/webexposed/rtcaudiojitterbuffermaxpackets-origin-trial-enabled.html
[modify] https://crrev.com/537539576bdbbdfcd46d93941b1a5ca9b57481fe/third_party/blink/public/platform/web_feature.mojom
[modify] https://crrev.com/537539576bdbbdfcd46d93941b1a5ca9b57481fe/third_party/blink/renderer/modules/peerconnection/rtc_configuration.idl
[modify] https://crrev.com/537539576bdbbdfcd46d93941b1a5ca9b57481fe/third_party/blink/renderer/modules/peerconnection/rtc_peer_connection.cc
[modify] https://crrev.com/537539576bdbbdfcd46d93941b1a5ca9b57481fe/third_party/blink/renderer/modules/peerconnection/rtc_peer_connection.h
[modify] https://crrev.com/537539576bdbbdfcd46d93941b1a5ca9b57481fe/third_party/blink/renderer/modules/peerconnection/rtc_peer_connection.idl
[modify] https://crrev.com/537539576bdbbdfcd46d93941b1a5ca9b57481fe/third_party/blink/renderer/platform/runtime_enabled_features.json5
[modify] https://crrev.com/537539576bdbbdfcd46d93941b1a5ca9b57481fe/tools/metrics/histograms/enums.xml

Project Member

Comment 2 by bugdroid1@chromium.org, Nov 21

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6c7d4b0c38ecea6c3cf1f5b1ac14bf353f897136

commit 6c7d4b0c38ecea6c3cf1f5b1ac14bf353f897136
Author: Jonas Olsson <jonasolsson@chromium.org>
Date: Wed Nov 21 12:48:05 2018

Add usage counter for WebRTC jitter buffer origin trial.

Bug: chromium:904764
Change-Id: I3ad24d066a2c60328921867b559fdd5e1a2dad00
Reviewed-on: https://chromium-review.googlesource.com/c/1346092
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Commit-Queue: Jonas Olsson <jonasolsson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610017}
[modify] https://crrev.com/6c7d4b0c38ecea6c3cf1f5b1ac14bf353f897136/third_party/blink/renderer/modules/peerconnection/rtc_peer_connection.cc

Sign in to add a comment