Migrated GN target :neteq_ilbc_quality_test

webrtc/modules/audio_coding/neteq/tools/audio_sink.cc

Index: webrtc/modules/audio_coding/neteq/tools/audio_sink.cc
diff --git a/webrtc/test/fuzzers/stun_validator_fuzzer.cc b/webrtc/modules/audio_coding/neteq/tools/audio_sink.cc
similarity index 52%
copy from webrtc/test/fuzzers/stun_validator_fuzzer.cc
copy to webrtc/modules/audio_coding/neteq/tools/audio_sink.cc
index 1f919f59dc58d4aa141cf94ac0f6a35e56be6b65..80e2ce3a25a2336fe0a784ac7d44da04500fb6f7 100644
--- a/webrtc/test/fuzzers/stun_validator_fuzzer.cc
+++ b/webrtc/modules/audio_coding/neteq/tools/audio_sink.cc
@@ -8,16 +8,16 @@
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
-#include <stddef.h>
-#include <stdint.h>
-
-#include "webrtc/p2p/base/stun.h"
+#include "webrtc/modules/audio_coding/neteq/tools/audio_sink.h"
 
 namespace webrtc {
-void FuzzOneInput(const uint8_t* data, size_t size) {
-  const char* message = reinterpret_cast<const char*>(data);
+namespace test {
 
-  cricket::StunMessage::ValidateFingerprint(message, size);
-  cricket::StunMessage::ValidateMessageIntegrity(message, size, "");
+// Moving this definition out fixes chromium-style errors: virtual

[hlundin-webrtc, 2016/08/10 11:33:14]

You can skip this comment. No need to document why the definition is in a .cc
file.

+// methods with non-empty bodies shouldn't be declared inline
+bool AudioSinkFork::WriteArray(const int16_t* audio, size_t num_samples) {
+  return left_sink_->WriteArray(audio, num_samples) &&
+         right_sink_->WriteArray(audio, num_samples);
 }
+}  // namespace test

[aleloi, 2016/08/10 10:14:01]

It's a new file. The diff is wrong. It has the method which chromium-style
complained about.

[hlundin-webrtc, 2016/08/10 11:33:15]

Acknowledged. It doesn't matter, but FYI, you can avoid finding these false
matches by using the --similarity option to git cl upload.

 }  // namespace webrtc