New issue
Advanced search Search tips

Issue 1987 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 17



Sign in to add a comment

Error code in ExecuteGet is not conformant with spec

Project Member Reported by kereliuk@chromium.org, Sep 8 2017

Issue description

Should be Invalid Argument and not Unknown Error.

spec: https://w3c.github.io/webdriver/webdriver-spec.html#navigate-to

 
Project Member

Comment 1 by johnchen@chromium.org, Aug 3

Labels: StandardsCompliance
Owner: ----
Since it's only the type of error that's wrong, I'm not marking this as blocking issue 2536.
Project Member

Comment 3 by johnchen@chromium.org, Sep 17

Labels: Pri-2
Owner: buldina@chromium.org
Status: Assigned (was: Available)
Labels: ToBeReleased ChromeDriver-2.43
Status: Fixed (was: Assigned)
Project Member

Comment 5 by johnchen@chromium.org, Sep 17

commit c263819ca8e3942de6387027a8686bad08cf1fc0
Author: Tatiana Buldina <buldina@chromium.org>
Date:   Mon Sep 17 21:06:47 2018 +0000

    [ChromeDriver] Update returned error codes
    
    Make ExecuteExecuteAsyncScript(), ExecuteGet(), ExecuteExecuteScript()
    return kInvalidArgument according W3C standard
    
    chromedriver: 2000, chromedriver:2001, chromedriver:1987
    Change-Id: Id5c82fca12bc32ef88c59e84f53d04f77e6114ca
    Reviewed-on: https://chromium-review.googlesource.com/1228753
    Commit-Queue: Tatiana Buldina <buldina@chromium.org>
    Reviewed-by: John Chen <johnchen@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#591813}
Project Member

Comment 6 by johnchen@chromium.org, Oct 17

Labels: -ToBeReleased

Sign in to add a comment