Jessica Jong [:jessica] (inactive) Reporter
	Description • 8 years ago

From bug 1271532 comment 17, 18 and 19.

In non-e10s <select>, to match the platform behavior in OS X and GTK, we should not update the combobox display text until the popup has been closed.

	Neil Deakin
	Comment 2 • 8 years ago

Comment on attachment 8786509 [details]
Bug 1275493 - Do not update <select> field while keying through items in dropdown with non-e10s on OS X and GTK.

Seems to work ok, but it should be Mac and Linux specific.

	Neil Deakin
	Comment 4 • 8 years ago
mozreview-review

Comment on attachment 8786509 [details]
Bug 1275493 - Do not update <select> field while keying through items in dropdown with non-e10s on OS X and GTK.

https://reviewboard.mozilla.org/r/75432/#review73960

	Pulsebot
	Comment 5 • 8 years ago

Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/75b2b455ddb1
Do not update <select> field while keying through items in dropdown with non-e10s on OS X and GTK. r=enndeakin+6102

	Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)
	Comment 6 • 8 years ago

Backed out for a11y selection failures e.g. in test_focus_selects.html:

https://hg.mozilla.org/integration/autoland/rev/03204f6c0a7a48b27187e4207e733e2abae8d538

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=75b2b455ddb166711e0875817c2dd1d143887b26
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=2915454&repo=autoland

14:52:38     INFO -  180 INFO Invoke the 'current item change for  'list' ' test { scenario #0: un }
14:52:38     INFO -  181 INFO TEST-PASS | chrome://mochitests/content/a11y/accessible/tests/mochitest/events/test_focus_selects.html | Test with ID = 'current item change for  'list' ' succeed. There's no unexpected focus event.
14:52:38     INFO -  182 INFO Invoke the 'current item change for  'combobox' ' test { scenario #0: un }
14:52:38     INFO -  183 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/tests/mochitest/events/test_focus_selects.html | Error in test: proposed current item 'cb_apple' is already current
14:52:38     INFO -  changeCurrentItem_reportError@chrome://mochitests/content/a11y/accessible/tests/mochitest/events.js:1384:5
14:52:38     INFO -  changeCurrentItem_invoke@chrome://mochitests/content/a11y/accessible/tests/mochitest/events.js:1348:9
14:52:38     INFO -  eventQueue_processNextInvoker@chrome://mochitests/content/a11y/accessible/tests/mochitest/events.js:438:9
14:52:38     INFO -  eventQueue/eventQueue_processNextInvokerInTimeout/<@chrome://mochitests/content/a11y/accessible/tests/mochitest/events.js:463:42
14:52:38     INFO -  setTimeout handler*SimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:622:12
14:52:38     INFO -  eventQueue_processNextInvokerInTimeout@chrome://mochitests/content/a11y/accessible/tests/mochitest/events.js:463:5
14:52:38     INFO -  eventQueue_processNextInvoker@chrome://mochitests/content/a11y/accessible/tests/mochitest/events.js:445:7
14:52:38     INFO -  eventQueue/eventQueue_processNextInvokerInTimeout/<@chrome://mochitests/content/a11y/accessible/tests/mochitest/events.js:458:43

	Mike Conley (:mconley) (:⚙️)
	Comment 7 • 8 years ago

Thanks for the backout, will come back around to this when I get a chance.