Fix Chromium clang plugin warnings

BUG=webrtc:163

Review-Url: https://codereview.webrtc.org/2285713002
Cr-Commit-Position: refs/heads/master@{#13943}

webrtc/modules/audio_processing/BUILD.gn

@@ -453,10 +453,6 @@ if (rtc_include_tests) {
       "//testing/gtest",
       "//third_party/gflags",
     ]
-    if (is_clang) {
-      # Suppress warnings from the Chromium Clang plugins (bugs.webrtc.org/163).
-      configs -= [ "//build/config/clang:find_bad_constructs" ]
-    }
   }
 
   executable("click_annotate") {

webrtc/modules/audio_processing/agc/agc.cc

@@ -97,4 +97,12 @@ int Agc::set_target_level_dbfs(int level) {
   return 0;
 }
 
+int Agc::target_level_dbfs() const {
+  return target_level_dbfs_;
+}
+
+float Agc::voice_probability() const {
+  return vad_.last_voice_probability();
+}
+
 }  // namespace webrtc

webrtc/modules/audio_processing/agc/agc.h

@@ -40,11 +40,8 @@ class Agc {
   virtual void Reset();
 
   virtual int set_target_level_dbfs(int level);
-  virtual int target_level_dbfs() const { return target_level_dbfs_; }
-
-  virtual float voice_probability() const {
-    return vad_.last_voice_probability();
-  }
+  virtual int target_level_dbfs() const;
+  virtual float voice_probability() const;
 
  private:
   double target_level_loudness_;

webrtc/modules/audio_processing/vad/voice_activity_detector.cc

@@ -31,6 +31,8 @@ VoiceActivityDetector::VoiceActivityDetector()
       standalone_vad_(StandaloneVad::Create()) {
 }
 
+VoiceActivityDetector::~VoiceActivityDetector() = default;
+
 // Because ISAC has a different chunk length, it updates
 // |chunkwise_voice_probabilities_| and |chunkwise_rms_| when there is new data.
 // Otherwise it clears them.

webrtc/modules/audio_processing/vad/voice_activity_detector.h

@@ -27,6 +27,7 @@ namespace webrtc {
 class VoiceActivityDetector {
  public:
   VoiceActivityDetector();
+  ~VoiceActivityDetector();
 
   // Processes each audio chunk and estimates the voice probability. The maximum
   // supported sample rate is 32kHz.