Migrated GN target :neteq_ilbc_quality_test

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

 /*
  *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  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