Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(702)

Unified Diff: content/browser/renderer_host/media/renderer_audio_output_stream_factory_context_impl_unittest.cc

Issue 2803853007: Ignore rounding error between clip_layer_length_ and scroll_layer_length_ (Closed)
Patch Set: expose IsFloatNearlyTheSame & add comment Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/test/layer_test_common.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/renderer_host/media/renderer_audio_output_stream_factory_context_impl_unittest.cc
diff --git a/content/browser/renderer_host/media/renderer_audio_output_stream_factory_context_impl_unittest.cc b/content/browser/renderer_host/media/renderer_audio_output_stream_factory_context_impl_unittest.cc
index 4a81273b7d168dc542e0543b494062594dbdf772..26ebf12c2ec00c8f7b9f259bbbd82b599e389dcb 100644
--- a/content/browser/renderer_host/media/renderer_audio_output_stream_factory_context_impl_unittest.cc
+++ b/content/browser/renderer_host/media/renderer_audio_output_stream_factory_context_impl_unittest.cc
@@ -185,7 +185,7 @@ class MockAudioOutputStream : public media::AudioOutputStream,
for (int frame = 0; frame < params.frames_per_buffer(); ++frame) {
// Using EXPECT here causes massive log spam in case of a broken test,
// and ASSERT causes it to hang, so we use CHECK.
- CHECK(cc::MathUtil::IsNearlyTheSameForTesting(
+ CHECK(cc::MathUtil::IsFloatNearlyTheSame(
expected_buffer->channel(0)[frame], dest->channel(0)[frame]))
<< "Got " << dest->channel(0)[frame] << ", expected "
<< expected_buffer->channel(0)[frame];
« no previous file with comments | « cc/test/layer_test_common.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698