New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 895211 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , iOS , Chrome , Mac , Fuchsia
Pri: 2
Type: Bug



Sign in to add a comment

Change buffer sizes types from signed to unsigned in audio code

Reported by andrew.macpherson@soundtrap.com, Oct 15

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/71.0.3574.0 Safari/537.36

Steps to reproduce the problem:
dcheng@ requested that an issue be filed to track cleanup of buffer size types in the audio code from signed to unsigned, the context for the discussion can be found here:

https://chromium-review.googlesource.com/c/chromium/src/+/1269527/5/media/base/ipc/media_param_traits.cc#90

What is the expected behavior?

What went wrong?

Did this work before? N/A 

Does this work in other browsers? N/A

Chrome version: 71.0.3574.0  Channel: n/a
OS Version: OS X 10.13.6
Flash Version:
 
Just realized that there's a typo in the title, 'times' should be 'types'. I'm not able to edit the issue though so just flagging it here.
Cc: maxmorin@chromium.org dalecur...@chromium.org olka@chromium.org
Components: Internals>Media>Audio
Labels: OS-Android OS-Chrome OS-Fuchsia OS-iOS OS-Linux OS-Windows
Summary: Change buffer sizes types from signed to unsigned in audio code (was: Change buffer sizes times from signed to unsigned in audio code)
Thanks Andrew.
Status: Available (was: Unconfirmed)

Sign in to add a comment