Commit de4ac0ee authored by Raymond Toy's avatar Raymond Toy Committed by Commit Bot

Remove method ComputeQHistogramValue()

The corresponding histograms were removed, but we forgot to remove
this bit of code.  Remove it.

Bug: 946853, 774526
Change-Id: Idfaf4f44083e97bb0082ada7f215ff3efb79a642
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542671Reviewed-by: default avatarHongchan Choi <hongchan@chromium.org>
Commit-Queue: Raymond Toy <rtoy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#646587}
parent b6de5573
...@@ -305,14 +305,6 @@ void AudioParamHandler::CalculateTimelineValues(float* values, ...@@ -305,14 +305,6 @@ void AudioParamHandler::CalculateTimelineValues(float* values,
sample_rate, sample_rate, MinValue(), MaxValue())); sample_rate, sample_rate, MinValue(), MaxValue()));
} }
int AudioParamHandler::ComputeQHistogramValue(float new_value) const {
// For the Q value, assume a useful range is [0, 25] and that 0.25 dB
// resolution is good enough. Then, we can map the floating point Q value (in
// dB) to an integer just by multipling by 4 and rounding.
new_value = clampTo(new_value, 0.0, 25.0);
return static_cast<int>(4 * new_value + 0.5);
}
// ---------------------------------------------------------------- // ----------------------------------------------------------------
AudioParam::AudioParam(BaseAudioContext& context, AudioParam::AudioParam(BaseAudioContext& context,
......
...@@ -216,8 +216,6 @@ class AudioParamHandler final : public ThreadSafeRefCounted<AudioParamHandler>, ...@@ -216,8 +216,6 @@ class AudioParamHandler final : public ThreadSafeRefCounted<AudioParamHandler>,
bool sample_accurate); bool sample_accurate);
void CalculateTimelineValues(float* values, unsigned number_of_values); void CalculateTimelineValues(float* values, unsigned number_of_values);
int ComputeQHistogramValue(float) const;
// The type of AudioParam, indicating what this AudioParam represents and what // The type of AudioParam, indicating what this AudioParam represents and what
// node it belongs to. Mostly for informational purposes and doesn't affect // node it belongs to. Mostly for informational purposes and doesn't affect
// implementation. // implementation.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment