Added missing TODOs in the beamformer unit test code.

webrtc/modules/audio_processing/beamformer/nonlinear_beamformer_unittest.cc

 /*
  *  Copyright (c) 2015 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 105 matching lines @@
   // Compare the output with the reference. Only the first values of the output
   // from last frame processed are compared in order not having to specify all
   // preceeding frames as testvectors. As the algorithm being tested has a
   // memory, testing only the last frame implicitly also tests the preceeding
   // frames.
   EXPECT_TRUE(test::BitExactFrame(
       capture_config.num_frames(), capture_config.num_channels(),
       output_reference, capture_output, kTolerance));
 }
 
+// TODO(peah): Add bitexactness tests for scenarios with more than 2 input
+// channels.
 std::vector<Point> CreateArrayGeometry(int variant) {
   std::vector<Point> array_geometry;
   switch (variant) {
     case 1:
       array_geometry.push_back(Point(-0.025f, 0.f, 0.f));
       array_geometry.push_back(Point(0.025f, 0.f, 0.f));
       break;
     case 2:
       array_geometry.push_back(Point(-0.035f, 0.f, 0.f));
       array_geometry.push_back(Point(0.035f, 0.f, 0.f));
@@ skipping 104 matching lines @@
                     bf.interf_angles_radians_[0]);
     EXPECT_FLOAT_EQ(M_PI / 2.f + bf.away_radians_,
                     bf.interf_angles_radians_[1]);
     bf.AimAt(AzimuthToSphericalPoint(bf.away_radians_ / 2.f));
     EXPECT_EQ(2u, bf.interf_angles_radians_.size());
     EXPECT_FLOAT_EQ(-bf.away_radians_ / 2.f, bf.interf_angles_radians_[0]);
     EXPECT_FLOAT_EQ(3.f * bf.away_radians_ / 2.f, bf.interf_angles_radians_[1]);
   }
 }
 
+// TODO(peah): Reenable all bitexactness tests once the division-by-zero issue
+// has been resolved.
+
 // TODO(peah): Investigate why the nonlinear_beamformer.cc causes a DCHECK in
 // this setup.
 TEST(BeamformerBitExactnessTest,
      DISABLED_Stereo8kHz_ArrayGeometry1_TargetDirection1) {
   const float kOutputReference[] = {0.001318f, -0.001091f, 0.000990f,
                                     0.001318f, -0.001091f, 0.000990f};
 
   RunBitExactnessTest(AudioProcessing::kSampleRate8kHz, CreateArrayGeometry(1),
                       TargetDirection1, kOutputReference);
 }
@@ skipping 104 matching lines @@
 TEST(BeamformerBitExactnessTest,
      DISABLED_Stereo16kHz_ArrayGeometry3_TargetDirection1) {
   const float kOutputReference[] = {-0.000161f, 0.000171f, -0.000096f,
                                     0.001007f,  0.000427f, 0.000977f};
 
   RunBitExactnessTest(AudioProcessing::kSampleRate16kHz, CreateArrayGeometry(3),
                       TargetDirection1, kOutputReference);
 }
 
 }  // namespace webrtc