New issue
Advanced search Search tips
Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 14



Sign in to add a comment
link

Issue 2003: Send Alert Text is not returning spec compliant error codes

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

Issue description

spec: https://w3c.github.io/webdriver/webdriver-spec.html#send-alert-text

We differ from the spec by at least the Unknown Error we return instead of Invalid Error
 

Comment 1 by vs495...@gmail.com, Dec 26 2017

Vishnu kumar

Comment 2 by johnchen@chromium.org, Aug 3

Project Member
Labels: StandardsCompliance

Comment 3 by foolip@chromium.org, Sep 14

Owner: johnchen@chromium.org
johnchen@, can you re-triage and find a new owner or leave available?

Comment 4 by johnchen@chromium.org, Sep 14

Project Member
Labels: Pri-2
Owner: buldina@chromium.org

Comment 5 by bugdroid1@chromium.org, Sep 14

Project Member
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/243f92ff54ff759a89328f699264ec078287a700

commit 243f92ff54ff759a89328f699264ec078287a700
Author: Tatiana Buldina <buldina@chromium.org>
Date: Fri Sep 14 19:24:18 2018

[ChromeDriver] Update error code of ExecuteSetAlertText()

Make ExecuteSetAlertText() return kInvalidArgument according W3C standard

Bug:  chromedriver:2003 
Change-Id: Ibd65e8861f0a3f17bb21b54b0d031b88de7dc05c
Reviewed-on: https://chromium-review.googlesource.com/1226794
Reviewed-by: John Chen <johnchen@chromium.org>
Commit-Queue: Tatiana Buldina <buldina@chromium.org>
Cr-Commit-Position: refs/heads/master@{#591428}
[modify] https://crrev.com/243f92ff54ff759a89328f699264ec078287a700/chrome/test/chromedriver/alert_commands.cc
[modify] https://crrev.com/243f92ff54ff759a89328f699264ec078287a700/docs/chromedriver_status.md

Comment 6 by buldina@chromium.org, Sep 14

Labels: ToBeReleased ChromeDriver-2.43
Status: Fixed (was: Available)

Comment 7 by johnchen@chromium.org, Oct 17

Project Member
Labels: -ToBeReleased

Sign in to add a comment