Remove OverUseDetectorOptions from OveruseDetector since it isn't used.

webrtc/common_types.h

Index: webrtc/common_types.h
diff --git a/webrtc/common_types.h b/webrtc/common_types.h
index 2988bf9f41a6fb8763cb31342217e24b4fe8d879..9d706b8ef7630359ad139cb5089ef2c8701e1c48 100644
--- a/webrtc/common_types.h
+++ b/webrtc/common_types.h
@@ -697,6 +697,9 @@ class BitrateAllocation {
 // Bandwidth over-use detector options.  These are used to drive
 // experimentation with bandwidth estimation parameters.
 // See modules/remote_bitrate_estimator/overuse_detector.h
+// TODO(terelius): This is only used in overuse_estimator.cc, and only in the
+// default constructed state. Can we move the relevant variables into that
+// class and delete this? See also disabled warning at line 27

[stefan-webrtc, 2016/12/22 09:37:25]

I would be happy to get rid of this struct. I've been close several times
before... :)

 struct OverUseDetectorOptions {
   OverUseDetectorOptions()
       : initial_slope(8.0 / 512.0),