Consider removing |render_frame_id| from AudioInputHostMsg_CreateStream and associated classes |
|||
Issue descriptionAudioInputHostMsg_CreateStream [1] doesn't need |render_frame_id| [2], which renders [3] unnecessary, except for the call to media internals [4], which is unfortunate. [1] https://code.google.com/p/chromium/codesearch#chromium/src/content/common/media/audio_messages.h&l=115&rcl=1456835474 [2] https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/renderer_host/media/audio_input_renderer_host.cc&l=352&q=audio_input_renderer_host.cc%20render_frame_id&sq=package:chromium [3] https://code.google.com/p/chromium/codesearch#chromium/src/content/renderer/media/audio_input_message_filter.cc&l=49&rcl=1456835474 [4] https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/renderer_host/media/audio_input_renderer_host.cc&sq=package:chromium&type=cs&l=502&rcl=1456835474
,
Mar 3 2017
This issue has been available for more than 365 days, and should be re-evaluated. Please re-triage this issue. The Hotlist-Recharge-Cold label is applied for tracking purposes, and should not be removed after re-triaging the issue. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 6 2017
This will go away when moving over to Mojo, which is in progress. Closing.
,
Mar 6 2017
grunell@ can you link to the mojo-bug-in-progress here? Thanks
,
Mar 6 2017
Swithing to Mojo is tracked in issue 653871 . |
|||
►
Sign in to add a comment |
|||
Comment 1 by tnakamura@chromium.org
, Mar 2 2016Status: Available (was: Unconfirmed)