Return from |Configure()| asynchronously, and allow |InitBuffers()| to return errors.

ppapi/tests/test_media_stream_audio_track.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // Tests PPB_MediaStreamAudioTrack interface.
 
 #include "ppapi/tests/test_media_stream_audio_track.h"
 
 #include "ppapi/c/private/ppb_testing_private.h"
 #include "ppapi/cpp/audio_buffer.h"
@@ skipping 186 matching lines @@
 std::string TestMediaStreamAudioTrack::TestConfigure() {
   // Create a track.
   instance_->EvalScript(kJSCode);
   event_.Wait();
   event_.Reset();
 
   ASSERT_FALSE(audio_track_.is_null());
   ASSERT_FALSE(audio_track_.HasEnded());
   ASSERT_FALSE(audio_track_.GetId().empty());
 
+  // Do nothing.

[teravest, 2014/07/31 20:54:55]

What's this comment for?

[Anand Mistry (off Chromium), 2014/07/31 23:54:48]

Expanded.

+  {
+    int32_t attrib_list[] = {
+      PP_MEDIASTREAMAUDIOTRACK_ATTRIB_NONE,
+    };
+    ASSERT_SUBTEST_SUCCESS(CheckConfigure(attrib_list, PP_OK));
+  }
+
   // Configure number of buffers.
   struct {
     int32_t buffers;
     int32_t expect_result;
   } buffers[] = {
     { 8, PP_OK },
     { 100, PP_OK },
     { kMaxNumberOfBuffers, PP_OK },
     { -1, PP_ERROR_BADARGUMENT },
     { kMaxNumberOfBuffers + 1, PP_OK },  // Clipped to max value.
@@ skipping 69 matching lines @@
     ASSERT_EQ(PP_ERROR_INPROGRESS, cc_configure.result());
     audio_track_.RecycleBuffer(buffer);
   }
 
   // Close the track.
   audio_track_.Close();
   ASSERT_TRUE(audio_track_.HasEnded());
   audio_track_ = pp::MediaStreamAudioTrack();
   PASS();
 }