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

Index: content/browser/media/webrtc_getusermedia_browsertest.cc
diff --git a/content/browser/media/webrtc_getusermedia_browsertest.cc b/content/browser/media/webrtc_getusermedia_browsertest.cc
index 945ab520c8adaa08efb9428e193719148d864a08..2006a2e03adc844154c4154e090d9bef22b96dcf 100644
--- a/content/browser/media/webrtc_getusermedia_browsertest.cc
+++ b/content/browser/media/webrtc_getusermedia_browsertest.cc
@@ -343,6 +343,27 @@ IN_PROC_BROWSER_TEST_F(WebRtcGetUserMediaBrowserTest, TwoGetUserMediaAndStop) {
       "twoGetUserMediaAndStop({video: true, audio: true});");
 }
 
+IN_PROC_BROWSER_TEST_F(WebRtcGetUserMediaBrowserTest,
+                       GetUserMediaWithTooHighVideoConstraintsValues) {
+  ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
+
+  GURL url(embedded_test_server()->GetURL("/media/getusermedia.html"));
+
+  int large_value = 99999;

[mcasas, 2014/03/13 08:07:46]

I'd recommend splitting this test in more navigate-check
for error parts, requesting a valid but not supported 
resolution, first, and then an unsupported frame rate --
this test runs on FakeVCD which will most likely never
support beyond 180p, 240p, 360p, 480p and 720p, so what
about 800x600? FakeVCD runs at 20fps (_not_ 30fps).

[perkj_chrome, 2014/03/14 08:29:52]

Discussed of line. This test is to test what happens if the constraints are
invalid. Leave as is.

+  std::string call = GenerateGetUserMediaCall(kGetUserMediaAndStop,
+                                              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.
+  ASSERT_EQ("Error: GetUserMedia call failed with code undefined",
+            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