Fix DCHECK error when invalid video constraints are set on gUM. If a video constraint can not be me…

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 277 matching lines @@
   ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
 
   std::vector<std::string> audio_ids;
   std::vector<std::string> video_ids;
   GetSources(&audio_ids, &video_ids);
 
   GURL url(embedded_test_server()->GetURL("/media/getusermedia.html"));
 
   // Test with invalid mandatory audio sourceID.
   NavigateToURL(shell(), url);
-  ExecuteJavascriptAndWaitForOk(
+  EXPECT_EQ("DeviceCaptureError", ExecuteJavascriptAndReturnResult(
       GenerateGetUserMediaWithMandatorySourceID(
           kGetUserMediaAndExpectFailure,
           "something invalid",
-          video_ids[0]));
+          video_ids[0])));
 
   // Test with invalid mandatory video sourceID.
-  ExecuteJavascriptAndWaitForOk(
+  EXPECT_EQ("DeviceCaptureError", ExecuteJavascriptAndReturnResult(
       GenerateGetUserMediaWithMandatorySourceID(
           kGetUserMediaAndExpectFailure,
           audio_ids[0],
-          "something invalid"));
+          "something invalid")));
 
   // Test with empty mandatory audio sourceID.
-  ExecuteJavascriptAndWaitForOk(
+  EXPECT_EQ("DeviceCaptureError", ExecuteJavascriptAndReturnResult(
       GenerateGetUserMediaWithMandatorySourceID(
           kGetUserMediaAndExpectFailure,
           "",
-          video_ids[0]));
+          video_ids[0])));
 }
 
 IN_PROC_BROWSER_TEST_P(WebRtcGetUserMediaBrowserTest,
                        GetUserMediaWithInvalidOptionalSourceID) {
   ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
 
   std::vector<std::string> audio_ids;
   std::vector<std::string> video_ids;
   GetSources(&audio_ids, &video_ids);
 
@@ skipping 25 matching lines @@
 IN_PROC_BROWSER_TEST_P(WebRtcGetUserMediaBrowserTest, TwoGetUserMediaAndStop) {
   ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
 
   GURL url(embedded_test_server()->GetURL("/media/getusermedia.html"));
   NavigateToURL(shell(), url);
 
   ExecuteJavascriptAndWaitForOk(
       "twoGetUserMediaAndStop({video: true, audio: true});");
 }
 
+IN_PROC_BROWSER_TEST_P(WebRtcGetUserMediaBrowserTest,
+                       GetUserMediaWithTooHighVideoConstraintsValues) {
+  ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
+
+  GURL url(embedded_test_server()->GetURL("/media/getusermedia.html"));
+
+  int large_value = 99999;
+  std::string call = GenerateGetUserMediaCall(kGetUserMediaAndExpectFailure,
+                                              large_value,
+                                              large_value,
+                                              large_value,
+                                              large_value,
+                                              large_value,
+                                              large_value);
+  NavigateToURL(shell(), url);
+
+  // TODO(perkj): A proper error code should be returned by gUM.
+  EXPECT_EQ("TrackStartError", ExecuteJavascriptAndReturnResult(call));
+}
+
 // 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_P(
     WebRtcGetUserMediaBrowserTest,
     TraceVideoCaptureControllerPerformanceDuringGetUserMedia) {
   RunGetUserMediaAndCollectMeasures(
       10,
       "VideoCaptureController::OnIncomingCapturedData",
@@ skipping 84 matching lines @@
     {640, 640, 480, 480, 30, 30},
     {960, 960, 720, 720, 30, 30},
     {1280, 1280, 720, 720, 30, 30},
     {1920, 1920, 1080, 1080, 30, 30}};
 
 INSTANTIATE_TEST_CASE_P(UserMedia,
                         WebRtcConstraintsBrowserTest,
                         testing::ValuesIn(kAllUserMediaSizes));
 
 }  // namespace content