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

Issue 787252 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
OOO Dec 22 - Jan 8
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task

Blocked on:
issue 653871



Sign in to add a comment

Mojofy and onion soup Media/Audio

Project Member Reported by slangley@chromium.org, Nov 21 2017

Issue description

- Convert IPCs to Mojo.
- Move code to modules/webaudio
- Remove redundant code as a result of onion souping.
 
slangley@ 

Can you be more specific? Which part of webaudio is problematic? 
- WebAudio codes are pretty much standalone, and they are already in modules/webaudio.
- We don't have any IPC call except for the initial handshaking for audio device.

Hi hongchang@

I see what looks like audio related files in content/renderer/media, for example audio_message_filter.cc and friends.

Are these not part of WebAudio? If not, can you suggest what the right component would be?

Thanks.
Cc: hongchan@chromium.org rtoy@chromium.org
Components: -Blink>WebAudio Internals>Media>Audio
Owner: olka@chromium.org
Summary: Mojofy and onion soup Media/Audio (was: Mojofy and onion soup WebAudio)
slangley@

That would be the internal Media/Audio team. There is already on-going work on this:
https://docs.google.com/document/d/1s_Fd1WRDdpb5n6C2MSJjeC3fis6hULZwfKMeDd4K5tI/edit

I believe olka@ is the right owner.

Comment 4 by olka@chromium.org, Dec 7 2017

Blockedon: 653871
Cc: guidou@chromium.org solenberg@chromium.org maxmorin@chromium.org
Converstion of audio stream IPCs to Mojo is in progress:
https://docs.google.com/document/d/1awQoajq_DLmz2AIU9iweC0zEYlVuHCvEIRQepeYyxh8/edit?usp=sharing

We are not planning any further work in content/renderer/media in the near future though.

slangley@ - what rearrangements are expected there? The only context I found is 
https://docs.google.com/document/d/1hBKAYGGM2Zm7qhqXtdLSubklVDKUo9ir-vJtPEOx11Y/edit#heading=h.yc28az2hj3n0

Comment 5 by olka@chromium.org, Dec 12 2017

Cc: olka@chromium.org
Owner: ----
Status: Available (was: Assigned)
Project Member

Comment 6 by sheriffbot@chromium.org, Dec 12

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Owner: olka@chromium.org
Status: Assigned (was: Untriaged)
The STO team has pretty much done all of this, so =>olka to close.
(WebAudio isn't part of this, but media/audio is)

Sign in to add a comment