Element Clear on a non-empty text field does not fire DOM change event

RESOLVED FIXED in Firefox 59

Status

defect
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: barancev, Assigned: ato)

Tracking

(Blocks 1 bug, {regression})

59 Branch
mozilla60
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox58 unaffected, firefox59 fixed, firefox60 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

a year ago
User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.108 Safari/537.36

Steps to reproduce:

Detected in Firefox Nightly 59.0a1 (2018-01-14) (64-bit)

Regression issue, may be caused by https://bugzilla.mozilla.org/show_bug.cgi?id=1354211

driver.get("https://output.jsbin.com/cafipek");
driver.findElement(By.id("clearMe")).clear();
assertEquals(driver.findElement(By.id("result")).getText(), "Cleared");


Actual results:

clear operation on a non-empty text field does not fire change event


Expected results:

clear operation should fire change event if it changes field value
Andreas, can you please have a look?
Blocks: 1354211
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(ato)
Keywords: regression
Summary: clear operation on a non-empty text field does not fire change event → Element Clear on a non-empty text field does not fire change event
Assignee

Comment 2

a year ago
This seems likely.  I will fix this after https://bugzilla.mozilla.org/show_bug.cgi?id=1430575.
Assignee: nobody → ato
Blocks: webdriver
Status: NEW → ASSIGNED
Depends on: 1430575
Flags: needinfo?(ato)
OS: Unspecified → All
Hardware: Unspecified → All
Summary: Element Clear on a non-empty text field does not fire change event → Element Clear on a non-empty text field does not fire DOM change event
Comment hidden (mozreview-request)

Comment 4

a year ago
mozreview-review
Comment on attachment 8945154 [details]
Bug 1430571 - Fire DOM change event on WebDriver:ElementClear.

https://reviewboard.mozilla.org/r/215396/#review221050
Attachment #8945154 - Flags: review?(dburns) → review+

Comment 5

a year ago
Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b5dc4c67afbb
Fire DOM change event on WebDriver:ElementClear. r=automatedtester

Comment 6

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b5dc4c67afbb
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
The test for this change looks stable so far. It would be good to get those changes uplifted to beta.
Whiteboard: [checkin-needed-beta]
Whiteboard: [checkin-needed-beta]
You need to log in before you can comment on or make changes to this bug.