1547009 - Perma open-features-non-integer-innerheight.html | features "innerheight=405.5" should set "height=405" - assert_equals: "top=0,left=0,width=401,innerheight=405.5 value after first non-digit will be ignored" when Gecko 68 merges to Beta on 2019-05-06

Status: RESOLVED DUPLICATE of bug 1540551

(Core :: DOM: Core & HTML, defect)

Description

[Raul Gurzau (:RaulG)]

[Tracking Requested - why for this release]:

Central as Beta simulation: https://treeherder.mozilla.org/#/jobs?repo=try&resultStatus=testfailed%2Cbusted%2Cexception&revision=689c102a69b154eac666dbd70df3a1d7beb60345&selectedJob=242589646

Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=242589646&repo=try&lineNumber=4458

Log snippet:

14:03:59 INFO - TEST-START | /html/browsers/the-window-object/apis-for-creating-and-navigating-browsing-contexts-by-name/open-features-non-integer-innerheight.html
14:03:59 INFO - Closing window 23622320129
14:03:59 INFO - PID 4688 | [Parent 1648, Gecko_IOThread] WARNING: pipe error: 109: file z:/task_1556189424/build/src/ipc/chromium/src/chrome/common/ipc_channel_win.cc, line 341
14:03:59 INFO - PID 4688 | [Child 3424, Chrome_ChildThread] WARNING: pipe error: 109: file z:/task_1556189424/build/src/ipc/chromium/src/chrome/common/ipc_channel_win.cc, line 341
14:03:59 INFO - PID 4688 | [Child 3424, Chrome_ChildThread] WARNING: pipe error: 109: file z:/task_1556189424/build/src/ipc/chromium/src/chrome/common/ipc_channel_win.cc, line 341
14:04:31 INFO -
14:04:31 INFO - TEST-PASS | /html/browsers/the-window-object/apis-for-creating-and-navigating-browsing-contexts-by-name/open-features-non-integer-innerheight.html | features "innerheight=/404" should NOT set "height=404"
14:04:31 INFO - TEST-PASS | /html/browsers/the-window-object/apis-for-creating-and-navigating-browsing-contexts-by-name/open-features-non-integer-innerheight.html | features "innerheight=_404" should NOT set "height=404"
14:04:31 INFO - TEST-PASS | /html/browsers/the-window-object/apis-for-creating-and-navigating-browsing-contexts-by-name/open-features-non-integer-innerheight.html | features "innerheight=L404" should NOT set "height=404"
14:04:31 INFO - TEST-UNEXPECTED-FAIL | /html/browsers/the-window-object/apis-for-creating-and-navigating-browsing-contexts-by-name/open-features-non-integer-innerheight.html | features "innerheight=405.5" should set "height=405" - assert_equals: "top=0,left=0,width=401,innerheight=405.5 value after first non-digit will be ignored" expected 405 but got 400
14:04:31 INFO - runWindowTests/</</<@http://web-platform.test:8000/html/browsers/the-window-object/apis-for-creating-and-navigating-browsing-contexts-by-name/open-features-non-integer-innerheight.html:68:9
14:04:31 INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1587:25
14:04:31 INFO - Test.prototype.step_func_done/<@http://web-platform.test:8000/resources/testharness.js:1627:32
14:04:31 INFO - PrefixedMessage.prototype.onMessage/<@http://web-platform.test:8000/common/PrefixedPostMessage.js:53:12
14:04:31 INFO -
14:04:31 INFO - TEST-UNEXPECTED-FAIL | /html/browsers/the-window-object/apis-for-creating-and-navigating-browsing-contexts-by-name/open-features-non-integer-innerheight.html | features "innerheight=405.32" should set "height=405" - assert_equals: "top=0,left=0,width=401,innerheight=405.32 value after first non-digit will be ignored" expected 405 but got 400
14:04:31 INFO - runWindowTests/</</<@http://web-platform.test:8000/html/browsers/the-window-object/apis-for-creating-and-navigating-browsing-contexts-by-name/open-features-non-integer-innerheight.html:68:9
14:04:31 INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1587:25
14:04:31 INFO - Test.prototype.step_func_done/<@http://web-platform.test:8000/resources/testharness.js:1627:32
14:04:31 INFO - PrefixedMessage.prototype.onMessage/<@http://web-platform.test:8000/common/PrefixedPostMessage.js:53:12
14:04:31 INFO -

Comment 1

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

I don't see how my patches could have broken this test. The only changes I made for that test was to slightly tweak the expectation metadata for android test runs.