New issue
Advanced search Search tips

Issue 854228 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on: View detail
issue 854229
issue 872412
issue 855758



Sign in to add a comment

[WebAudio refactoring] project tracker

Project Member Reported by hongchan@chromium.org, Jun 19 2018

Issue description

This is a tracking bug of WebAudio (modules/webaudio, platform/audio) refactoring.
 

Comment 1 Deleted

Blockedon: 854229

Comment 3 Deleted

Comment 4 Deleted

Blockedon: 855758
Project Member

Comment 6 by bugdroid1@chromium.org, Jul 11

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

commit f3e11cf3cb81cc08f9bb78c5063f370cd21c8059
Author: Hongchan Choi <hongchan@chromium.org>
Date: Wed Jul 11 21:12:56 2018

Clean up AudioDestinationNode

1. Add and clarify comments.
2. Reorder methods for the readability.
  (WebIDL, Inherited actions, Own actions, Getters,
   Member variables)

Bug: 854228
Test: Passes all layout tests.
Change-Id: I47e27e86fa17272fc2cce1c0b8bb85c8da8b1ea6
Reviewed-on: https://chromium-review.googlesource.com/1124924
Reviewed-by: Raymond Toy <rtoy@chromium.org>
Commit-Queue: Hongchan Choi <hongchan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#574337}
[modify] https://crrev.com/f3e11cf3cb81cc08f9bb78c5063f370cd21c8059/third_party/blink/renderer/modules/webaudio/audio_destination_node.h

Blockedon: 872412
The priority of refactoring Web Audio API is quite low, also it only has two issues at this point. Not sure if having a tracker issue is appropriate.

Sign in to add a comment