New issue
Advanced search Search tips

Issue 718554 link

Starred by 1 user
Status: Verified
Owner:
hongchan@chromium.org
Closed: May 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 469639



Sign in to add a comment

Adding trace events in WebAudio rendering pipe line

Project Member Reported by hongchan@chromium.org, May 4 2017 
We need a better profiling system for the WebAudio rendering. Adding multiple trace events (TRACE_EVENTX) in various points in the rendering pipe line would be helpful:

 - device-callback
 - device-posttask
 - webthread-render-start
 - webthread-render-end
Project Member

Comment 1 by bugdroid1@chromium.org, May 5 2017 

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

commit ab52b62684a4634d2d5e7fdc2a8a137af9aa8647
Author: hongchan <hongchan@chromium.org>
Date: Fri May 05 01:06:12 2017

Add TRACE_EVENTX() in WebAudio rendering pipe line

This CL adds multiple trace events (TRACE_EVENTX) in various points in
the rendering to investigate the cause of glitches.

("webaudio", "AudioDestination::Start")
("webaudio", "AudioDestination::Stop")
("webaudio", "AudioDestination::RequestRenderOnWebThread", "frames_to_render", frames_to_render)
("webaudio", "AudioDestination::Render", "callback_buffer_size", number_of_frames)
("webaudio", "AudioDestinationHandler::Render")

BUG= 718554 

Review-Url: https://codereview.chromium.org/2858223003
Cr-Commit-Position: refs/heads/master@{#469557}

[modify] https://crrev.com/ab52b62684a4634d2d5e7fdc2a8a137af9aa8647/third_party/WebKit/Source/modules/webaudio/AudioDestinationNode.cpp
[modify] https://crrev.com/ab52b62684a4634d2d5e7fdc2a8a137af9aa8647/third_party/WebKit/Source/platform/audio/AudioDestination.cpp

Comment 2 by hongchan@chromium.org, May 8 2017

Status: Verified (was: Started)
►

Sign in to add a comment