[webkit-changes] [219833] trunk/Source/WebDriver

Mon, 24 Jul 2017 12:17:10 -0700

Title: [219833] trunk/Source/WebDriver
Revision
219833
Author
[email protected]
Date
2017-07-24 12:16:55 -0700 (Mon, 24 Jul 2017)

Log Message

WebDriver: rename m_browsingContext as m_currentBrowsingContext in Session
https://bugs.webkit.org/show_bug.cgi?id=174783

Patch by Carlos Garcia Campos <[email protected]> on 2017-07-24
Reviewed by Brian Burg.

We have m_toplevelBrowsingContext and m_browsingContext, which is confusing. m_browsingContext is actually the
current browsing context, and the spec also refers to it as the current browsing context, so better use
m_currentBrowsingContext.

* Session.cpp:
(WebDriver::Session::switchToTopLevelBrowsingContext):
(WebDriver::Session::switchToBrowsingContext):
(WebDriver::Session::switchToFrame):
(WebDriver::Session::switchToParentFrame):
(WebDriver::Session::computeElementLayout):
(WebDriver::Session::findElements):
(WebDriver::Session::isElementSelected):
(WebDriver::Session::getElementText):
(WebDriver::Session::getElementTagName):
(WebDriver::Session::isElementEnabled):
(WebDriver::Session::isElementDisplayed):
(WebDriver::Session::getElementAttribute):
(WebDriver::Session::waitForNavigationToComplete):
(WebDriver::Session::elementClear):
(WebDriver::Session::elementSendKeys):
(WebDriver::Session::elementSubmit):
(WebDriver::Session::executeScript):
* Session.h:

Modified Paths

Diff

Modified: trunk/Source/WebDriver/ChangeLog (219832 => 219833)


--- trunk/Source/WebDriver/ChangeLog	2017-07-24 18:55:29 UTC (rev 219832)
+++ trunk/Source/WebDriver/ChangeLog	2017-07-24 19:16:55 UTC (rev 219833)
@@ -1,3 +1,34 @@
+2017-07-24  Carlos Garcia Campos  <[email protected]>
+
+        WebDriver: rename m_browsingContext as m_currentBrowsingContext in Session
+        https://bugs.webkit.org/show_bug.cgi?id=174783
+
+        Reviewed by Brian Burg.
+
+        We have m_toplevelBrowsingContext and m_browsingContext, which is confusing. m_browsingContext is actually the
+        current browsing context, and the spec also refers to it as the current browsing context, so better use
+        m_currentBrowsingContext.
+
+        * Session.cpp:
+        (WebDriver::Session::switchToTopLevelBrowsingContext):
+        (WebDriver::Session::switchToBrowsingContext):
+        (WebDriver::Session::switchToFrame):
+        (WebDriver::Session::switchToParentFrame):
+        (WebDriver::Session::computeElementLayout):
+        (WebDriver::Session::findElements):
+        (WebDriver::Session::isElementSelected):
+        (WebDriver::Session::getElementText):
+        (WebDriver::Session::getElementTagName):
+        (WebDriver::Session::isElementEnabled):
+        (WebDriver::Session::isElementDisplayed):
+        (WebDriver::Session::getElementAttribute):
+        (WebDriver::Session::waitForNavigationToComplete):
+        (WebDriver::Session::elementClear):
+        (WebDriver::Session::elementSendKeys):
+        (WebDriver::Session::elementSubmit):
+        (WebDriver::Session::executeScript):
+        * Session.h:
+
 2017-07-20  Carlos Garcia Campos  <[email protected]>
 
         WebDriver: implement page load timeout

Modified: trunk/Source/WebDriver/Session.cpp (219832 => 219833)


--- trunk/Source/WebDriver/Session.cpp	2017-07-24 18:55:29 UTC (rev 219832)
+++ trunk/Source/WebDriver/Session.cpp	2017-07-24 19:16:55 UTC (rev 219833)
@@ -90,7 +90,7 @@
 void Session::switchToTopLevelBrowsingContext(std::optional<String> toplevelBrowsingContext)
 {
     m_toplevelBrowsingContext = toplevelBrowsingContext;
-    m_browsingContext = std::nullopt;
+    m_currentBrowsingContext = std::nullopt;
 }
 
 void Session::switchToBrowsingContext(std::optional<String> browsingContext)
@@ -97,9 +97,9 @@
 {
     // Automation sends empty strings for main frame.
     if (!browsingContext || browsingContext.value().isEmpty())
-        m_browsingContext = std::nullopt;
+        m_currentBrowsingContext = std::nullopt;
     else
-        m_browsingContext = browsingContext;
+        m_currentBrowsingContext = browsingContext;
 }
 
 void Session::createTopLevelBrowsingContext(Function<void (CommandResult&&)>&& completionHandler)
@@ -370,8 +370,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
 
     int frameIndex;
     if (frameID->asInteger(frameIndex)) {
@@ -416,7 +416,7 @@
         return;
     }
 
-    if (!m_browsingContext) {
+    if (!m_currentBrowsingContext) {
         completionHandler(CommandResult::success());
         return;
     }
@@ -423,7 +423,7 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     m_host->sendCommandToBackend(ASCIILiteral("resolveParentFrameHandle"), WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
         if (response.isError || !response.responseObject) {
             completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
@@ -586,7 +586,7 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     parameters->setString(ASCIILiteral("nodeHandle"), elementID);
     parameters->setBoolean(ASCIILiteral("scrollIntoViewIfNeeded"), options.contains(ElementLayoutOption::ScrollIntoViewIfNeeded));
     parameters->setBoolean(ASCIILiteral("useViewportCoordinates"), options.contains(ElementLayoutOption::UseViewportCoordinates));
@@ -653,8 +653,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     parameters->setString(ASCIILiteral("function"), FindNodesJavaScript);
     parameters->setArray(ASCIILiteral("arguments"), WTFMove(arguments));
     parameters->setBoolean(ASCIILiteral("expectsImplicitCallbackArgument"), true);
@@ -720,8 +720,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     parameters->setString(ASCIILiteral("function"), ElementAttributeJavaScript);
     parameters->setArray(ASCIILiteral("arguments"), WTFMove(arguments));
     m_host->sendCommandToBackend(ASCIILiteral("evaluateJavaScriptFunction"), WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
@@ -764,8 +764,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     // FIXME: Add an atom to properly implement this instead of just using innerText.
     parameters->setString(ASCIILiteral("function"), ASCIILiteral("function(element) { return element.innerText.replace(/^[^\\S\\xa0]+|[^\\S\\xa0]+$/g, '') }"));
     parameters->setArray(ASCIILiteral("arguments"), WTFMove(arguments));
@@ -800,8 +800,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     parameters->setString(ASCIILiteral("function"), ASCIILiteral("function(element) { return element.tagName.toLowerCase() }"));
     parameters->setArray(ASCIILiteral("arguments"), WTFMove(arguments));
     m_host->sendCommandToBackend(ASCIILiteral("evaluateJavaScriptFunction"), WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
@@ -856,8 +856,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     parameters->setString(ASCIILiteral("function"), ASCIILiteral("function(element) { return element.disabled === undefined ? true : !element.disabled }"));
     parameters->setArray(ASCIILiteral("arguments"), WTFMove(arguments));
     m_host->sendCommandToBackend(ASCIILiteral("evaluateJavaScriptFunction"), WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
@@ -891,8 +891,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     parameters->setString(ASCIILiteral("function"), ElementDisplayedJavaScript);
     parameters->setArray(ASCIILiteral("arguments"), WTFMove(arguments));
     m_host->sendCommandToBackend(ASCIILiteral("evaluateJavaScriptFunction"), WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
@@ -927,8 +927,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     parameters->setString(ASCIILiteral("function"), ElementAttributeJavaScript);
     parameters->setArray(ASCIILiteral("arguments"), WTFMove(arguments));
     m_host->sendCommandToBackend(ASCIILiteral("evaluateJavaScriptFunction"), WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
@@ -959,8 +959,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     if (m_timeouts.pageLoad)
         parameters->setInteger(ASCIILiteral("pageLoadTimeout"), m_timeouts.pageLoad.value().millisecondsAs<int>());
     m_host->sendCommandToBackend(ASCIILiteral("waitForNavigationToComplete"), WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
@@ -1013,8 +1013,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     parameters->setString(ASCIILiteral("function"), FormElementClearJavaScript);
     parameters->setArray(ASCIILiteral("arguments"), WTFMove(arguments));
     m_host->sendCommandToBackend(ASCIILiteral("evaluateJavaScriptFunction"), WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
@@ -1175,8 +1175,8 @@
     arguments->pushString(createElement(elementID)->toJSONString());
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     parameters->setString(ASCIILiteral("function"), focusScript);
     parameters->setArray(ASCIILiteral("arguments"), WTFMove(arguments));
     m_host->sendCommandToBackend(ASCIILiteral("evaluateJavaScriptFunction"), WTFMove(parameters), [this, protectedThis = makeRef(*this), keys = WTFMove(keys), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) mutable {
@@ -1234,8 +1234,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     parameters->setString(ASCIILiteral("function"), FormSubmitJavaScript);
     parameters->setArray(ASCIILiteral("arguments"), WTFMove(arguments));
     m_host->sendCommandToBackend(ASCIILiteral("evaluateJavaScriptFunction"), WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
@@ -1293,8 +1293,8 @@
 
     RefPtr<InspectorObject> parameters = InspectorObject::create();
     parameters->setString(ASCIILiteral("browsingContextHandle"), m_toplevelBrowsingContext.value());
-    if (m_browsingContext)
-        parameters->setString(ASCIILiteral("frameHandle"), m_browsingContext.value());
+    if (m_currentBrowsingContext)
+        parameters->setString(ASCIILiteral("frameHandle"), m_currentBrowsingContext.value());
     parameters->setString(ASCIILiteral("function"), "function(){" + script + '}');
     parameters->setArray(ASCIILiteral("arguments"), WTFMove(arguments));
     if (mode == ExecuteScriptMode::Async) {

Modified: trunk/Source/WebDriver/Session.h (219832 => 219833)


--- trunk/Source/WebDriver/Session.h	2017-07-24 18:55:29 UTC (rev 219832)
+++ trunk/Source/WebDriver/Session.h	2017-07-24 19:16:55 UTC (rev 219833)
@@ -159,7 +159,7 @@
     Timeouts m_timeouts;
     String m_id;
     std::optional<String> m_toplevelBrowsingContext;
-    std::optional<String> m_browsingContext;
+    std::optional<String> m_currentBrowsingContext;
 };
 
 } // WebDriver

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to