Title: [224090] trunk/Source/WebDriver
Revision
224090
Author
[email protected]
Date
2017-10-27 00:13:11 -0700 (Fri, 27 Oct 2017)

Log Message

WebDriver: unknown capabilities should produce an InvalidArgument error when validating capabilities
https://bugs.webkit.org/show_bug.cgi?id=178868

Reviewed by Brian Burg.

7.2 Processing Capabilities
https://w3c.github.io/webdriver/webdriver-spec.html#dfn-validate-capabilities

Fixes: imported/w3c/webdriver/tests/sessions/new_session/invalid_capabilities.py::test_invalid_extensions

* WebDriverService.cpp:
(WebDriver::WebDriverService::validatedCapabilities const):

Modified Paths

Diff

Modified: trunk/Source/WebDriver/ChangeLog (224089 => 224090)


--- trunk/Source/WebDriver/ChangeLog	2017-10-27 07:11:56 UTC (rev 224089)
+++ trunk/Source/WebDriver/ChangeLog	2017-10-27 07:13:11 UTC (rev 224090)
@@ -1,5 +1,20 @@
 2017-10-27  Carlos Garcia Campos  <[email protected]>
 
+        WebDriver: unknown capabilities should produce an InvalidArgument error when validating capabilities
+        https://bugs.webkit.org/show_bug.cgi?id=178868
+
+        Reviewed by Brian Burg.
+
+        7.2 Processing Capabilities
+        https://w3c.github.io/webdriver/webdriver-spec.html#dfn-validate-capabilities
+
+        Fixes: imported/w3c/webdriver/tests/sessions/new_session/invalid_capabilities.py::test_invalid_extensions
+
+        * WebDriverService.cpp:
+        (WebDriver::WebDriverService::validatedCapabilities const):
+
+2017-10-27  Carlos Garcia Campos  <[email protected]>
+
         WebDriver: deserializeTimeouts should fail if the value is not integer
         https://bugs.webkit.org/show_bug.cgi?id=178866
 

Modified: trunk/Source/WebDriver/WebDriverService.cpp (224089 => 224090)


--- trunk/Source/WebDriver/WebDriverService.cpp	2017-10-27 07:11:56 UTC (rev 224089)
+++ trunk/Source/WebDriver/WebDriverService.cpp	2017-10-27 07:13:11 UTC (rev 224090)
@@ -404,7 +404,8 @@
             if (!platformValidateCapability(it->key, it->value))
                 return nullptr;
             result->setValue(it->key, RefPtr<InspectorValue>(it->value));
-        }
+        } else
+            return nullptr;
     }
     return result;
 }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to