Delete webrtc/base/common.h

webrtc/modules/remote_bitrate_estimator/test/bwe_test_framework.h

Index: webrtc/modules/remote_bitrate_estimator/test/bwe_test_framework.h
diff --git a/webrtc/modules/remote_bitrate_estimator/test/bwe_test_framework.h b/webrtc/modules/remote_bitrate_estimator/test/bwe_test_framework.h
index 1ae24af965e0db4bf0069ff70d915f15748ecb77..698e765bdbd2aabe349d4ddf45fef46629575dea 100644
--- a/webrtc/modules/remote_bitrate_estimator/test/bwe_test_framework.h
+++ b/webrtc/modules/remote_bitrate_estimator/test/bwe_test_framework.h
@@ -250,7 +250,7 @@ class RateCounterFilter : public PacketProcessor {
   Stats<double> packets_per_second_stats_;
   Stats<double> kbps_stats_;
   int64_t start_plotting_time_ms_;
-  int flow_id_;
+  int flow_id_ ATTRIBUTE_UNUSED;

[kwiberg-webrtc, 2017/02/09 09:48:43]

Why not remove it if it's unused? (Or if it's used conditionally, #ifdef this
line on the same condition?)

I'm asking because I suspect we can manage to not have an UNUSED macro at all.

[nisse-webrtc, 2017/02/09 11:26:18]

Changed to #if. It's mentioned in two constructors. One can be replaced with a
default value here, the other needs another #ifdef.

   std::string name_;
   // Algorithm name if single flow, Total link utilization if all flows.
   std::string algorithm_name_;