Henrik Skupin [:whimboo][⌚️UTC+1] Assignee
	Description • 6 years ago

Because we only type when keys exist the default value will remain, and it is not possible to clear it.

To make this possible the input field needs to be cleared first.

	Henrik Skupin [:whimboo][⌚️UTC+1] Assignee
	Comment 1 • 6 years ago

Here the WebDriver spec definition of setting the text:

> Perform user agent dependent steps to set the value of current user prompt’s text field to text.

As such internally Marionette uses `interaction.sendKeysToElement()` similar to the `Element Send Keys` command. Hereby the latter command doesn't replace the content, but concatenates the given `text` at the end.

	Henrik Skupin [:whimboo][⌚️UTC+1] Assignee
	Comment 3 • 6 years ago

The patch refactors the test file, and also adds an additional test,
which makes sure that the default text of the prompt is replaced in
case of an empty string is send as text.

	Henrik Skupin [:whimboo][⌚️UTC+1] Assignee
	Comment 4 • 6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b98d46fabd5a0bb6afe3ffccf736752f9a0879b5

	Pulsebot
	Comment 5 • 6 years ago

Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f3ea8e988254
[marionette] Replace and not append text for "WebDriver:SendAlertText". r=jgraham
https://hg.mozilla.org/integration/mozilla-inbound/rev/82dc592c7293
[wdspec] Add test for empty text to send_alert_text/send.py. r=jgraham

	Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)
	Comment 6 • 6 years ago

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/12735 for changes under testing/web-platform/tests

	Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)
	Comment 7 • 6 years ago

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

	Cosmin Sabou [:CosminS]
	Comment 8 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/f3ea8e988254
https://hg.mozilla.org/mozilla-central/rev/82dc592c7293

	Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)
	Comment 9 • 6 years ago

Upstream PR merged