Content browser test that checks gum callback is fired after failure

content/browser/media/webrtc_getusermedia_browsertest.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.
 
 #include "base/command_line.h"
 #include "base/debug/trace_event_impl.h"
 #include "base/json/json_reader.h"
 #include "base/strings/stringprintf.h"
 #include "base/test/trace_event_analyzer.h"
 #include "base/values.h"
@@ skipping 483 matching lines @@
                                               large_value,
                                               large_value,
                                               large_value,
                                               large_value);
   NavigateToURL(shell(), url);
 
   EXPECT_EQ("ConstraintNotSatisfiedError",
             ExecuteJavascriptAndReturnResult(call));
 }
 
+// This test will make two getUserMedia requests, first one triggers error

[phoglund_chromium, 2014/10/15 18:03:09]

Nit: phrase better; what about
"This test makes two getUserMedia requests, one with impossible constraints that
should trigger an error, and one with valid constraints. The test verifies
getUserMedia can succeed after being given impossible constraints."

[jansson, 2014/10/16 07:08:33]

Done.

+// callback while the 2nd one triggers success callback. Goal is to check that
+// callbacks are still fired after an error callback has fired.
+IN_PROC_BROWSER_TEST_F(WebRtcGetUserMediaBrowserTest,
+                       TwoGetUserMediaAndCheckCallbackAfterFailure) {
+  ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
+
+  GURL url(embedded_test_server()->GetURL("/media/getusermedia.html"));
+  NavigateToURL(shell(), url);
+
+  int large_value = 99999;
+  std::string constraints1 =

[no longer working on chromium, 2014/10/15 14:31:54]

nit const std::string 

[phoglund_chromium, 2014/10/15 18:03:09]

Nit: Calling it constraints1|2 is misleading, since it's really a getUserMedia
call populated with parameters (one of which is the constraints). A better name
would gum_with_impossible_constraints and gum_with_vga_constraints.

[jansson, 2014/10/16 07:08:33]

Done.

[jansson, 2014/10/16 07:08:33]

Done.

+    GenerateGetUserMediaCall(kGetUserMediaAndExpectFailure,
+                             large_value,
+                             large_value,
+                             large_value,
+                             large_value,
+                             large_value,
+                             large_value);
+  std::string constraints2 =

[no longer working on chromium, 2014/10/15 14:31:54]

ditto

[jansson, 2014/10/16 07:08:33]

Done.

+    GenerateGetUserMediaCall(kGetUserMediaAndAnalyseAndStop,
+                             640, 640, 480, 480, 10, 30);
+
+  ASSERT_EQ("ConstraintNotSatisfiedError",
+            ExecuteJavascriptAndReturnResult(constraints1));
+
+  ASSERT_EQ("w=640:h=480",
+            ExecuteJavascriptAndReturnResult(constraints2));
+}
+
 // This test will make a simple getUserMedia page, verify that video is playing
 // in a simple local <video>, and for a couple of seconds, collect some
 // performance traces from VideoCaptureController colorspace conversion and
 // potential resizing.
 IN_PROC_BROWSER_TEST_F(
     WebRtcGetUserMediaBrowserTest,
     TraceVideoCaptureControllerPerformanceDuringGetUserMedia) {
   RunGetUserMediaAndCollectMeasures(
       10,
       "VideoCaptureController::OnIncomingCapturedData",
@@ skipping 93 matching lines @@
     {640, 640, 360, 360, 10, 30},
     {640, 640, 480, 480, 10, 30},
     {960, 960, 720, 720, 10, 30},
     {1280, 1280, 720, 720, 10, 30}};
 
 INSTANTIATE_TEST_CASE_P(UserMedia,
                         WebRtcConstraintsBrowserTest,
                         testing::ValuesIn(kAllUserMediaSizes));
 
 }  // namespace content