Reducing the number of const casts and fixing constness of accessors improves const correctness and code readability.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/18408fcc6f0e2410d7b91f420d468e629eff6731 commit 18408fcc6f0e2410d7b91f420d468e629eff6731 Author: Eero Häkkinen <eero.hakkinen@intel.com> Date: Fri Mar 16 15:30:40 2018 Remove unneeded const casts from VectorMath functions Bug: 822721 Change-Id: If0d5768d344580ce7da3b8cf07eaacece34614eb Reviewed-on: https://chromium-review.googlesource.com/962141 Commit-Queue: Eero Häkkinen <eero.hakkinen@intel.com> Reviewed-by: Raymond Toy <rtoy@chromium.org> Cr-Commit-Position: refs/heads/master@{#543706} [modify] https://crrev.com/18408fcc6f0e2410d7b91f420d468e629eff6731/third_party/WebKit/Source/platform/audio/mac/VectorMathMac.h
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4f6f6ba63ba317144d1a01599b0fefeef580aeac commit 4f6f6ba63ba317144d1a01599b0fefeef580aeac Author: Eero Häkkinen <eero.hakkinen@intel.com> Date: Mon Mar 19 16:10:59 2018 Remove unneeded const casts from AudioBus and FFTFrame This also improves const correctness of FFTFrame. Bug: 822721 Change-Id: I49668b37b0ecca278ce35af293eee6e324d88f4c Reviewed-on: https://chromium-review.googlesource.com/962223 Reviewed-by: Raymond Toy <rtoy@chromium.org> Commit-Queue: Eero Häkkinen <eero.hakkinen@intel.com> Cr-Commit-Position: refs/heads/master@{#544053} [modify] https://crrev.com/4f6f6ba63ba317144d1a01599b0fefeef580aeac/third_party/WebKit/Source/modules/webaudio/RealtimeAnalyser.cpp [modify] https://crrev.com/4f6f6ba63ba317144d1a01599b0fefeef580aeac/third_party/WebKit/Source/platform/audio/AudioBus.cpp [modify] https://crrev.com/4f6f6ba63ba317144d1a01599b0fefeef580aeac/third_party/WebKit/Source/platform/audio/AudioBus.h [modify] https://crrev.com/4f6f6ba63ba317144d1a01599b0fefeef580aeac/third_party/WebKit/Source/platform/audio/FFTConvolver.cpp [modify] https://crrev.com/4f6f6ba63ba317144d1a01599b0fefeef580aeac/third_party/WebKit/Source/platform/audio/FFTConvolver.h [modify] https://crrev.com/4f6f6ba63ba317144d1a01599b0fefeef580aeac/third_party/WebKit/Source/platform/audio/FFTFrame.cpp [modify] https://crrev.com/4f6f6ba63ba317144d1a01599b0fefeef580aeac/third_party/WebKit/Source/platform/audio/FFTFrame.h [modify] https://crrev.com/4f6f6ba63ba317144d1a01599b0fefeef580aeac/third_party/WebKit/Source/platform/audio/ffmpeg/FFTFrameFFMPEG.cpp
There is not any unneeded const casts under third_party/WebKit/Source/platform/audio anymore.
Comment 1 by bugdroid1@chromium.org
, Mar 16 2018