Removing some unnecessary string manipulation code from VoEBase::GetVersion().

webrtc/voice_engine/voe_base_unittest.cc

 /*
  *  Copyright (c) 2013 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 60 matching lines @@
   EXPECT_EQ(0, base_->DeleteChannel(channel_1));
   // Make sure that the only use of the channel-to-delete is |reference|
   // at this point.
   EXPECT_EQ(1, reference.use_count());
 
   reference = shared_data->channel_manager().GetChannel(channel_2);
   EXPECT_EQ(0, base_->DeleteChannel(channel_2));
   EXPECT_EQ(1, reference.use_count());
 }
 
+TEST_F(VoEBaseTest, GetVersion) {
+  char v1[1024] = {0};

[kwiberg-webrtc, 2015/12/02 10:50:12]

Or fill it with something other than zeros, to test if GetVersion really appends
a null.

+  base_->GetVersion(v1);
+  std::string v2 = VoiceEngine::GetVersionString() + "\n";
+  EXPECT_EQ(v2, v1);
+}
 }  // namespace webrtc