New issue
Advanced search Search tips

Issue 1995 link

Starred by 10 users

Issue metadata

Status: Verified
Owner:
Closed: Nov 6


Show other hotlists

Hotlists containing this issue:
Hotlist-3


Sign in to add a comment

XML special case of Is Element Enabled is not handled as per spec

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

Issue description

This should always return false if the document is of type "xml" as per spec: https://w3c.github.io/webdriver/webdriver-spec.html#is-element-enabled

However, we currently don't handle this.
 
Project Member

Comment 1 by kereliuk@chromium.org, Jul 26

Owner: ----
Status: Untriaged (was: Unconfirmed)
Project Member

Comment 2 by johnchen@chromium.org, Jul 31

Labels: StandardsCompliance Pri-2
Status: Available (was: Untriaged)
Project Member

Comment 3 by johnchen@chromium.org, Oct 17

Owner: buldina@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 4 by johnchen@chromium.org, Oct 17

The following Python code snippet can repro this issue:

from selenium import webdriver
driver = webdriver.Chrome()
driver.get('https://www.w3schools.com/xml/note.xml')
print(driver.find_element_by_tag_name('body').is_enabled())

With current ChromeDriver (up to version 2.43), the last statement prints True, while a correct implementation should print False.
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 6

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/61453ebecb8e38603d1a57e7fca7da0b42dd18b4

commit 61453ebecb8e38603d1a57e7fca7da0b42dd18b4
Author: Tatiana Buldina <buldina@chromium.org>
Date: Tue Nov 06 00:35:46 2018

[ChromeDriver] XML special case of Is Element Enabled

Bug:  chromedriver:1995 
Change-Id: I471db88d118c8954421c2d360a277e4f15c0e8b2
Reviewed-on: https://chromium-review.googlesource.com/c/1318810
Reviewed-by: John Chen <johnchen@chromium.org>
Commit-Queue: Tatiana Buldina <buldina@chromium.org>
Cr-Commit-Position: refs/heads/master@{#605538}
[modify] https://crrev.com/61453ebecb8e38603d1a57e7fca7da0b42dd18b4/chrome/test/chromedriver/element_commands.cc

Labels: ToBeReleased ChromeDriver-2.44
Status: Fixed (was: Assigned)
Project Member

Comment 7 by johnchen@chromium.org, Nov 6

Status: Verified (was: Fixed)

Sign in to add a comment