Fix up comments in WebAudio to match code |
|||
Issue descriptionIn The Great Blink Rename, many things were renamed, but the corresponding comments were not touched. We need to update the comments to match the code.
,
Jul 3 2017
@rtoy, I'm interesting in this work. Can you let me complete? I have some question. 1. Should capitalized function names are fixed in comments? eg. pullInputs() -> PullInputs() 2. Is it enough fixing under webaudio folder or not?
,
Jul 7 2017
I think there is a couple of things around this issue: 1. Variable names: m_foo -> |foo_| 2. Method names: fooBar() -> |FooBar()| 3. We manage both modules/audio and platform/audio. Thanks for your interest and let me know if you have more questions!
,
Jul 9
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
,
Jul 17
Closing (WontFix, obsolete) File new separate bugs if and when the comments are updated. |
|||
►
Sign in to add a comment |
|||
Comment 1 by bugdroid1@chromium.org
, May 11 2017