Title: [225739] trunk/Source/WebDriver
- Revision
- 225739
- Author
- [email protected]
- Date
- 2017-12-11 00:30:25 -0800 (Mon, 11 Dec 2017)
Log Message
WebDriver: get active element should return no such element error when there isn't an active element
https://bugs.webkit.org/show_bug.cgi?id=180421
Reviewed by Brian Burg.
We currently return unknown error.
Fixes: imported/w3c/webdriver/tests/element_retrieval/get_active_element.py::test_missing_document_element
* Session.cpp:
(WebDriver::Session::getActiveElement):
Modified Paths
Diff
Modified: trunk/Source/WebDriver/ChangeLog (225738 => 225739)
--- trunk/Source/WebDriver/ChangeLog 2017-12-11 07:55:10 UTC (rev 225738)
+++ trunk/Source/WebDriver/ChangeLog 2017-12-11 08:30:25 UTC (rev 225739)
@@ -1,3 +1,17 @@
+2017-12-11 Carlos Garcia Campos <[email protected]>
+
+ WebDriver: get active element should return no such element error when there isn't an active element
+ https://bugs.webkit.org/show_bug.cgi?id=180421
+
+ Reviewed by Brian Burg.
+
+ We currently return unknown error.
+
+ Fixes: imported/w3c/webdriver/tests/element_retrieval/get_active_element.py::test_missing_document_element
+
+ * Session.cpp:
+ (WebDriver::Session::getActiveElement):
+
2017-12-04 Brian Burg <[email protected]>
Web Automation: add flag to preserve legacy page screenshot behavior
Modified: trunk/Source/WebDriver/Session.cpp (225738 => 225739)
--- trunk/Source/WebDriver/Session.cpp 2017-12-11 07:55:10 UTC (rev 225738)
+++ trunk/Source/WebDriver/Session.cpp 2017-12-11 08:30:25 UTC (rev 225739)
@@ -996,7 +996,7 @@
}
RefPtr<JSON::Object> elementObject = createElement(WTFMove(resultValue));
if (!elementObject) {
- completionHandler(CommandResult::fail(CommandResult::ErrorCode::UnknownError));
+ completionHandler(CommandResult::fail(CommandResult::ErrorCode::NoSuchElement));
return;
}
completionHandler(CommandResult::success(WTFMove(elementObject)));
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
