Removed the issue with the leading semicolon in the audio processing module experiment description…

webrtc/modules/audio_processing/echo_cancellation_impl.cc

 /*
  *  Copyright (c) 2012 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 399 matching lines @@
 
 bool EchoCancellationImpl::is_aec3_enabled() const {
   rtc::CritScope cs(crit_capture_);
   return aec3_enabled_;
 }
 
 std::string EchoCancellationImpl::GetExperimentsDescription() {
   rtc::CritScope cs(crit_capture_);
   std::string description = (aec3_enabled_ ? "AEC3" : "");
   if (refined_adaptive_filter_enabled_) {
-    description += ";RefinedAdaptiveFilter";
+    if (description.size() > 0) {

[hlundin-webrtc, 2016/04/19 06:49:33]

Isn't it better to have a semi-colon separated list that always ends the last
item with a semi-colon? That way, each component would just have to make sure to
add a semi-colon after the string(s) it inserts. And if it inserts nothing, no
semi-colon is added either.

[peah-webrtc, 2016/04/19 07:55:12]

Awesome variant! :-).

Done.

+      description += ";";
+    }
+    description += "RefinedAdaptiveFilter";
   }
   return description;
 }
 
 bool EchoCancellationImpl::is_refined_adaptive_filter_enabled() const {
   rtc::CritScope cs(crit_capture_);
   return refined_adaptive_filter_enabled_;
 }
 
 bool EchoCancellationImpl::is_extended_filter_enabled() const {
@@ skipping 147 matching lines @@
   return error;
 }
 
 size_t EchoCancellationImpl::NumCancellersRequired() const {
   RTC_DCHECK(stream_properties_);
   return stream_properties_->num_output_channels *
          stream_properties_->num_reverse_channels;
 }
 
 }  // namespace webrtc