⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Changeset 238662 in webkit


Ignore:
Timestamp:
Nov 29, 2018, 1:14:12 AM (8 years ago)
Author:
tpopela@redhat.com
Message:

[WebDriver] Fix clang warnings in Session.cpp
https://bugs.webkit.org/show_bug.cgi?id=192077

Reviewed by Carlos Garcia Campos.

  • Session.cpp:

(WebDriver::Session::dismissAndNotifyAlert):
(WebDriver::Session::acceptAndNotifyAlert):
(WebDriver::Session::reportUnexpectedAlertOpen):
(WebDriver::Session::getCurrentURL):
(WebDriver::Session::getTitle):
(WebDriver::Session::getWindowHandle):
(WebDriver::Session::getWindowHandles):
(WebDriver::Session::getToplevelBrowsingContextRect):
(WebDriver::Session::computeElementLayout):
(WebDriver::Session::isElementSelected):
(WebDriver::Session::getElementText):
(WebDriver::Session::getElementTagName):
(WebDriver::Session::getElementRect):
(WebDriver::Session::isElementEnabled):
(WebDriver::Session::isElementDisplayed):
(WebDriver::Session::getElementAttribute):
(WebDriver::Session::getElementProperty):
(WebDriver::Session::getElementCSSValue):
(WebDriver::Session::selectOptionElement):
(WebDriver::Session::elementClick):
(WebDriver::Session::elementClear):
(WebDriver::Session::handleScriptResult):
(WebDriver::Session::performMouseInteraction):
(WebDriver::Session::performKeyboardInteractions):
(WebDriver::Session::getAllCookies):
(WebDriver::Session::getNamedCookie):
(WebDriver::Session::addCookie):
(WebDriver::Session::deleteCookie):
(WebDriver::Session::deleteAllCookies):
(WebDriver::Session::performActions):
(WebDriver::Session::releaseActions):
(WebDriver::Session::dismissAlert):
(WebDriver::Session::acceptAlert):
(WebDriver::Session::getAlertText):
(WebDriver::Session::sendAlertText):
(WebDriver::Session::takeScreenshot):

  • WebDriverService.cpp:

(WebDriver::WebDriverService::deleteSession):

Location:
trunk/Source/WebDriver
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebDriver/ChangeLog

    r237724 r238662  
     12018-11-29  Tomas Popela  <tpopela@redhat.com>
     2
     3        [WebDriver] Fix clang warnings in Session.cpp
     4        https://bugs.webkit.org/show_bug.cgi?id=192077
     5
     6        Reviewed by Carlos Garcia Campos.
     7
     8        * Session.cpp:
     9        (WebDriver::Session::dismissAndNotifyAlert):
     10        (WebDriver::Session::acceptAndNotifyAlert):
     11        (WebDriver::Session::reportUnexpectedAlertOpen):
     12        (WebDriver::Session::getCurrentURL):
     13        (WebDriver::Session::getTitle):
     14        (WebDriver::Session::getWindowHandle):
     15        (WebDriver::Session::getWindowHandles):
     16        (WebDriver::Session::getToplevelBrowsingContextRect):
     17        (WebDriver::Session::computeElementLayout):
     18        (WebDriver::Session::isElementSelected):
     19        (WebDriver::Session::getElementText):
     20        (WebDriver::Session::getElementTagName):
     21        (WebDriver::Session::getElementRect):
     22        (WebDriver::Session::isElementEnabled):
     23        (WebDriver::Session::isElementDisplayed):
     24        (WebDriver::Session::getElementAttribute):
     25        (WebDriver::Session::getElementProperty):
     26        (WebDriver::Session::getElementCSSValue):
     27        (WebDriver::Session::selectOptionElement):
     28        (WebDriver::Session::elementClick):
     29        (WebDriver::Session::elementClear):
     30        (WebDriver::Session::handleScriptResult):
     31        (WebDriver::Session::performMouseInteraction):
     32        (WebDriver::Session::performKeyboardInteractions):
     33        (WebDriver::Session::getAllCookies):
     34        (WebDriver::Session::getNamedCookie):
     35        (WebDriver::Session::addCookie):
     36        (WebDriver::Session::deleteCookie):
     37        (WebDriver::Session::deleteAllCookies):
     38        (WebDriver::Session::performActions):
     39        (WebDriver::Session::releaseActions):
     40        (WebDriver::Session::dismissAlert):
     41        (WebDriver::Session::acceptAlert):
     42        (WebDriver::Session::getAlertText):
     43        (WebDriver::Session::sendAlertText):
     44        (WebDriver::Session::takeScreenshot):
     45        * WebDriverService.cpp:
     46        (WebDriver::WebDriverService::deleteSession):
     47
    1482018-11-02  Zan Dobersek  <zdobersek@igalia.com>
    249
  • trunk/Source/WebDriver/Session.cpp

    r235560 r238662  
    242242{
    243243    reportUnexpectedAlertOpen([this, completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
    244         dismissAlert([this, errorResult = WTFMove(result), completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
     244        dismissAlert([errorResult = WTFMove(result), completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
    245245            if (result.isError()) {
    246246                completionHandler(WTFMove(result));
     
    255255{
    256256    reportUnexpectedAlertOpen([this, completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
    257         acceptAlert([this, errorResult = WTFMove(result), completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
     257        acceptAlert([errorResult = WTFMove(result), completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
    258258            if (result.isError()) {
    259259                completionHandler(WTFMove(result));
     
    267267void Session::reportUnexpectedAlertOpen(Function<void (CommandResult&&)>&& completionHandler)
    268268{
    269     getAlertText([this, completionHandler = WTFMove(completionHandler)](CommandResult&& result) {
     269    getAlertText([completionHandler = WTFMove(completionHandler)](CommandResult&& result) {
    270270        std::optional<String> alertText;
    271271        if (!result.isError()) {
     
    329329        RefPtr<JSON::Object> parameters = JSON::Object::create();
    330330        parameters->setString("handle"_s, m_toplevelBrowsingContext.value());
    331         m_host->sendCommandToBackend("getBrowsingContext"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     331        m_host->sendCommandToBackend("getBrowsingContext"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    332332            if (response.isError || !response.responseObject) {
    333333                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    449449        parameters->setString("function"_s, "function() { return document.title; }"_s);
    450450        parameters->setArray("arguments"_s, JSON::Array::create());
    451         m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     451        m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    452452            if (response.isError || !response.responseObject) {
    453453                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    478478    RefPtr<JSON::Object> parameters = JSON::Object::create();
    479479    parameters->setString("handle"_s, m_toplevelBrowsingContext.value());
    480     m_host->sendCommandToBackend("getBrowsingContext"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     480    m_host->sendCommandToBackend("getBrowsingContext"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    481481        if (response.isError || !response.responseObject) {
    482482            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    556556void Session::getWindowHandles(Function<void (CommandResult&&)>&& completionHandler)
    557557{
    558     m_host->sendCommandToBackend("getBrowsingContexts"_s, JSON::Object::create(), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     558    m_host->sendCommandToBackend("getBrowsingContexts"_s, JSON::Object::create(), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    559559        if (response.isError || !response.responseObject) {
    560560            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    687687    RefPtr<JSON::Object> parameters = JSON::Object::create();
    688688    parameters->setString("handle"_s, m_toplevelBrowsingContext.value());
    689     m_host->sendCommandToBackend("getBrowsingContext"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     689    m_host->sendCommandToBackend("getBrowsingContext"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    690690        if (response.isError || !response.responseObject) {
    691691            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    916916    parameters->setBoolean("scrollIntoViewIfNeeded"_s, options.contains(ElementLayoutOption::ScrollIntoViewIfNeeded));
    917917    parameters->setString("coordinateSystem"_s, options.contains(ElementLayoutOption::UseViewportCoordinates) ? "LayoutViewport"_s : "Page"_s);
    918     m_host->sendCommandToBackend("computeElementLayout"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) mutable {
     918    m_host->sendCommandToBackend("computeElementLayout"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) mutable {
    919919        if (response.isError || !response.responseObject) {
    920920            completionHandler(std::nullopt, std::nullopt, false, WTFMove(response.responseObject));
     
    11131113        parameters->setString("function"_s, ElementAttributeJavaScript);
    11141114        parameters->setArray("arguments"_s, WTFMove(arguments));
    1115         m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1115        m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    11161116            if (response.isError || !response.responseObject) {
    11171117                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    11641164        parameters->setString("function"_s, "function(element) { return element.innerText.replace(/^[^\\S\\xa0]+|[^\\S\\xa0]+$/g, '') }"_s);
    11651165        parameters->setArray("arguments"_s, WTFMove(arguments));
    1166         m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1166        m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    11671167            if (response.isError || !response.responseObject) {
    11681168                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    12051205        parameters->setString("function"_s, "function(element) { return element.tagName.toLowerCase() }"_s);
    12061206        parameters->setArray("arguments"_s, WTFMove(arguments));
    1207         m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1207        m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    12081208            if (response.isError || !response.responseObject) {
    12091209                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    12371237            return;
    12381238        }
    1239         computeElementLayout(elementID, { }, [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](std::optional<Rect>&& rect, std::optional<Point>&&, bool, RefPtr<JSON::Object>&& error) {
     1239        computeElementLayout(elementID, { }, [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](std::optional<Rect>&& rect, std::optional<Point>&&, bool, RefPtr<JSON::Object>&& error) {
    12401240            if (!rect || error) {
    12411241                completionHandler(CommandResult::fail(WTFMove(error)));
     
    12731273        parameters->setString("function"_s, "function(element) { return element.disabled === undefined ? true : !element.disabled }"_s);
    12741274        parameters->setArray("arguments"_s, WTFMove(arguments));
    1275         m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1275        m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    12761276            if (response.isError || !response.responseObject) {
    12771277                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    13141314        parameters->setString("function"_s, ElementDisplayedJavaScript);
    13151315        parameters->setArray("arguments"_s, WTFMove(arguments));
    1316         m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1316        m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    13171317            if (response.isError || !response.responseObject) {
    13181318                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    13561356        parameters->setString("function"_s, ElementAttributeJavaScript);
    13571357        parameters->setArray("arguments"_s, WTFMove(arguments));
    1358         m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1358        m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    13591359            if (response.isError || !response.responseObject) {
    13601360                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    13971397        parameters->setString("function"_s, makeString("function(element) { return element.", property, "; }"));
    13981398        parameters->setArray("arguments"_s, WTFMove(arguments));
    1399         m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1399        m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    14001400            if (response.isError || !response.responseObject) {
    14011401                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    14381438        parameters->setString("function"_s, makeString("function(element) { return document.defaultView.getComputedStyle(element).getPropertyValue('", cssProperty, "'); }"));
    14391439        parameters->setArray("arguments"_s, WTFMove(arguments));
    1440         m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1440        m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    14411441            if (response.isError || !response.responseObject) {
    14421442                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    14991499    parameters->setString("frameHandle"_s, m_currentBrowsingContext.value_or(emptyString()));
    15001500    parameters->setString("nodeHandle"_s, elementID);
    1501     m_host->sendCommandToBackend("selectOptionElement"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1501    m_host->sendCommandToBackend("selectOptionElement"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    15021502        if (response.isError) {
    15031503            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    15301530        }
    15311531
    1532         getElementTagName(elementID, [this, elementID, inViewCenter = WTFMove(inViewCenter), isObscured, completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
     1532        getElementTagName(elementID, [this, elementID, inViewCenter = WTFMove(inViewCenter), completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
    15331533            bool isOptionElement = false;
    15341534            if (!result.isError()) {
     
    15701570    parameters->setString("function"_s, FormElementClearJavaScript);
    15711571    parameters->setArray("arguments"_s, WTFMove(arguments));
    1572     m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1572    m_host->sendCommandToBackend("evaluateJavaScriptFunction"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    15731573        if (response.isError) {
    15741574            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    18221822    }
    18231823
    1824     return resultValue;
     1824    return WTFMove(resultValue);
    18251825}
    18261826
     
    19261926    }
    19271927    parameters->setArray("modifiers"_s, JSON::Array::create());
    1928     m_host->sendCommandToBackend("performMouseInteraction"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1928    m_host->sendCommandToBackend("performMouseInteraction"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    19291929        if (response.isError) {
    19301930            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    19601960    }
    19611961    parameters->setArray("interactions"_s, WTFMove(interactionsArray));
    1962     m_host->sendCommandToBackend("performKeyboardInteractions"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     1962    m_host->sendCommandToBackend("performKeyboardInteractions"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    19631963        if (response.isError) {
    19641964            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    20472047        RefPtr<JSON::Object> parameters = JSON::Object::create();
    20482048        parameters->setString("browsingContextHandle"_s, m_toplevelBrowsingContext.value());
    2049         m_host->sendCommandToBackend("getAllCookies"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) mutable {
     2049        m_host->sendCommandToBackend("getAllCookies"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) mutable {
    20502050            if (response.isError || !response.responseObject) {
    20512051                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    20802080void Session::getNamedCookie(const String& name, Function<void (CommandResult&&)>&& completionHandler)
    20812081{
    2082     getAllCookies([this, name, completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
     2082    getAllCookies([name, completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
    20832083        if (result.isError()) {
    20842084            completionHandler(WTFMove(result));
     
    21172117        parameters->setString("browsingContextHandle"_s, m_toplevelBrowsingContext.value());
    21182118        parameters->setObject("cookie"_s, WTFMove(cookie));
    2119         m_host->sendCommandToBackend("addSingleCookie"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     2119        m_host->sendCommandToBackend("addSingleCookie"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    21202120            if (response.isError) {
    21212121                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    21422142        parameters->setString("browsingContextHandle"_s, m_toplevelBrowsingContext.value());
    21432143        parameters->setString("cookieName"_s, name);
    2144         m_host->sendCommandToBackend("deleteSingleCookie"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     2144        m_host->sendCommandToBackend("deleteSingleCookie"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    21452145            if (response.isError) {
    21462146                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    21662166        RefPtr<JSON::Object> parameters = JSON::Object::create();
    21672167        parameters->setString("browsingContextHandle"_s, m_toplevelBrowsingContext.value());
    2168         m_host->sendCommandToBackend("deleteAllCookies"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     2168        m_host->sendCommandToBackend("deleteAllCookies"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    21692169            if (response.isError) {
    21702170                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    23382338
    23392339        parameters->setArray("steps"_s, WTFMove(steps));
    2340         m_host->sendCommandToBackend("performInteractionSequence"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)] (SessionHost::CommandResponse&& response) {
     2340        m_host->sendCommandToBackend("performInteractionSequence"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)] (SessionHost::CommandResponse&& response) {
    23412341            if (response.isError) {
    23422342                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    23602360    RefPtr<JSON::Object> parameters = JSON::Object::create();
    23612361    parameters->setString("handle"_s, m_toplevelBrowsingContext.value());
    2362     m_host->sendCommandToBackend("cancelInteractionSequence"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     2362    m_host->sendCommandToBackend("cancelInteractionSequence"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    23632363        if (response.isError) {
    23642364            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    23782378    RefPtr<JSON::Object> parameters = JSON::Object::create();
    23792379    parameters->setString("browsingContextHandle"_s, m_toplevelBrowsingContext.value());
    2380     m_host->sendCommandToBackend("dismissCurrentJavaScriptDialog"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     2380    m_host->sendCommandToBackend("dismissCurrentJavaScriptDialog"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    23812381        if (response.isError) {
    23822382            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    23962396    RefPtr<JSON::Object> parameters = JSON::Object::create();
    23972397    parameters->setString("browsingContextHandle"_s, m_toplevelBrowsingContext.value());
    2398     m_host->sendCommandToBackend("acceptCurrentJavaScriptDialog"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     2398    m_host->sendCommandToBackend("acceptCurrentJavaScriptDialog"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    23992399        if (response.isError) {
    24002400            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    24142414    RefPtr<JSON::Object> parameters = JSON::Object::create();
    24152415    parameters->setString("browsingContextHandle"_s, m_toplevelBrowsingContext.value());
    2416     m_host->sendCommandToBackend("messageOfCurrentJavaScriptDialog"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     2416    m_host->sendCommandToBackend("messageOfCurrentJavaScriptDialog"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    24172417        if (response.isError || !response.responseObject) {
    24182418            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    24382438    parameters->setString("browsingContextHandle"_s, m_toplevelBrowsingContext.value());
    24392439    parameters->setString("userInput"_s, text);
    2440     m_host->sendCommandToBackend("setUserInputForCurrentJavaScriptPrompt"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
     2440    m_host->sendCommandToBackend("setUserInputForCurrentJavaScriptPrompt"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) {
    24412441        if (response.isError) {
    24422442            completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
     
    24692469        if (scrollIntoView.value_or(false))
    24702470            parameters->setBoolean("scrollIntoViewIfNeeded"_s, true);
    2471         m_host->sendCommandToBackend("takeScreenshot"_s, WTFMove(parameters), [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) mutable {
     2471        m_host->sendCommandToBackend("takeScreenshot"_s, WTFMove(parameters), [protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)](SessionHost::CommandResponse&& response) mutable {
    24722472            if (response.isError || !response.responseObject) {
    24732473                completionHandler(CommandResult::fail(WTFMove(response.responseObject)));
  • trunk/Source/WebDriver/WebDriverService.cpp

    r234837 r238662  
    759759
    760760    auto session = std::exchange(m_session, nullptr);
    761     session->close([this, session, completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
     761    session->close([session, completionHandler = WTFMove(completionHandler)](CommandResult&& result) mutable {
    762762        // Ignore unknown errors when closing the session if the browser is closed.
    763763        if (result.isError() && result.errorCode() == CommandResult::ErrorCode::UnknownError && !session->isConnected())
Note: See TracChangeset for help on using the changeset viewer.