Fix Chromium clang plugin warnings

webrtc/common.h

Index: webrtc/common.h
diff --git a/webrtc/common.h b/webrtc/common.h
index 8795064fd07e3ebb0e63e0a51cdb2ee8e7b1cc6b..79db50c5b5ad9f9be7bf1809e5e647f8913bab0d 100644
--- a/webrtc/common.h
+++ b/webrtc/common.h
@@ -71,15 +71,8 @@ class Config {
   // This instance gets ownership of the newly set value.
   template<typename T> void Set(T* value);
 
-  Config() {}
-  ~Config() {
-    // Note: this method is inline so webrtc public API depends only
-    // on the headers.

[kwiberg-webrtc, 2016/08/29 08:28:18]

I had to remove this comment, because it's not true anymore. Hopefully it wasn't
vital...

[kjellander_webrtc, 2016/08/29 19:18:09]

Right, if moving this breaks anything it's the right thing to do since targets
should properly depend on targets with .cc files, not just including headers
blindly.

-    for (OptionMap::iterator it = options_.begin();
-         it != options_.end(); ++it) {
-      delete it->second;
-    }
-  }
+  Config();
+  ~Config();
 
  private:
   struct BaseOption {