Address some violations of chromium-style.

webrtc/modules/congestion_controller/delay_based_bwe.h

 /*
  *  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.
  */
 
@@ skipping 27 matching lines @@
   struct Result {
     Result() : updated(false), probe(false), target_bitrate_bps(0) {}
     Result(bool probe, uint32_t target_bitrate_bps)
         : updated(true), probe(probe), target_bitrate_bps(target_bitrate_bps) {}
     bool updated;
     bool probe;
     uint32_t target_bitrate_bps;
   };
 
   DelayBasedBwe(RtcEventLog* event_log, const Clock* clock);
-  virtual ~DelayBasedBwe() {}

[kwiberg-webrtc, 2017/05/31 08:22:27]

Hmm. Why is this destructor virtual? Nothing else in this class seems to be.

[nisse-webrtc, 2017/05/31 08:44:54]

No idea, I'm afraid. Do you want me to change it?

[kwiberg-webrtc, 2017/05/31 08:49:48]

No, at least not in this CL. I was just asking in case you already knew.

+  virtual ~DelayBasedBwe();
 
   Result IncomingPacketFeedbackVector(
       const std::vector<PacketFeedback>& packet_feedback_vector);
   void OnRttUpdate(int64_t avg_rtt_ms, int64_t max_rtt_ms);
   bool LatestEstimate(std::vector<uint32_t>* ssrcs,
                       uint32_t* bitrate_bps) const;
   void SetStartBitrate(int start_bitrate_bps);
   void SetMinBitrate(int min_bitrate_bps);
   int64_t GetExpectedBwePeriodMs() const;
 
@@ skipping 47 matching lines @@
   uint32_t last_logged_bitrate_;
   BandwidthUsage last_logged_state_;
   bool in_sparse_update_experiment_;
 
   RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(DelayBasedBwe);
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_CONGESTION_CONTROLLER_DELAY_BASED_BWE_H_