Closed Bug 1705193 Opened 4 years ago Closed 4 years ago

Categories

(Core :: Layout, defect, P5)

defect

Tracking

()

RESOLVED FIXED
90 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox87 --- unaffected
firefox88 --- unaffected
firefox89 --- fixed
firefox90 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: abutkovits)

References

(Regression)

Details

(Keywords: intermittent-failure, regression)

Attachments

(3 files)

Filed by: nbeleuzu [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=336539158&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/cjxHn-cGT0OEPLIqOisvKg/runs/0/artifacts/public/logs/live_backing.log
Reftest URL: https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/cjxHn-cGT0OEPLIqOisvKg/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1


[task 2021-04-14T19:50:28.015Z] 19:50:28     INFO - PID 1614 | [Child 1626, Main Thread] WARNING: Shouldn't call SchedulePaint in a detached pres context: file /builds/worker/checkouts/gecko/layout/generic/nsIFrame.cpp:7138
[task 2021-04-14T19:50:28.027Z] 19:50:28     INFO - PID 1614 | 1618429828025	Marionette	INFO	Testing http://web-platform.test:8000/css/css-ui/text-overflow-022.html == http://web-platform.test:8000/css/css-ui/reference/text-overflow-022-ref.html
[task 2021-04-14T19:50:28.050Z] 19:50:28     INFO - PID 1643 | [Child 1649, Main Thread] WARNING: nsAppShell::Exit() called redundantly: file /builds/worker/checkouts/gecko/widget/cocoa/nsAppShell.mm:751
[task 2021-04-14T19:50:28.053Z] 19:50:28     INFO - PID 1643 | ###!!! [Parent][RunMessage] Error: Channel closing: too late to send/recv, messages will be lost
[task 2021-04-14T19:50:28.059Z] 19:50:28     INFO - PID 1643 | ###!!! [Parent][RunMessage] Error: Channel closing: too late to send/recv, messages will be lost
[task 2021-04-14T19:50:28.061Z] 19:50:28     INFO - PID 1643 | [Child 1650, Main Thread] WARNING: nsAppShell::Exit() called redundantly: file /builds/worker/checkouts/gecko/widget/cocoa/nsAppShell.mm:751
[task 2021-04-14T19:50:28.062Z] 19:50:28     INFO - PID 1643 | ###!!! [Parent][RunMessage] Error: Channel closing: too late to send/recv, messages will be lost
[task 2021-04-14T19:50:28.103Z] 19:50:28     INFO - PID 1643 | [2021-04-14T19:50:28Z WARN  xulstore::persist] tried to remove key that isn't in the store
[task 2021-04-14T19:50:28.104Z] 19:50:28     INFO - PID 1643 | [2021-04-14T19:50:28Z WARN  xulstore::persist] tried to remove key that isn't in the store
[task 2021-04-14T19:50:28.116Z] 19:50:28     INFO - PID 1643 | [Child 1650, Main Thread] WARNING: Extra shutdown CC: 'i < NORMAL_SHUTDOWN_COLLECTIONS', file /builds/worker/checkouts/gecko/xpcom/base/nsCycleCollector.cpp:3361
[task 2021-04-14T19:50:28.118Z] 19:50:28     INFO - PID 1643 | [Child 1650, Main Thread] WARNING: '!obs', file /builds/worker/checkouts/gecko/toolkit/components/sessionstore/RestoreTabContentObserver.cpp:58
[task 2021-04-14T19:50:28.127Z] 19:50:28     INFO - PID 1643 | [Child 1650, Main Thread] WARNING: NS_ENSURE_TRUE(Preferences::InitStaticMembers()) failed: file /builds/worker/checkouts/gecko/modules/libpref/Preferences.cpp:4442
[task 2021-04-14T19:50:28.175Z] 19:50:28     INFO - PID 1614 | 1618429828173	Marionette	INFO	Allowed 0-12 pixels different, maximum difference per channel 172-172
[task 2021-04-14T19:50:28.213Z] 19:50:28     INFO - PID 1643 | [Child 1649, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80040111 (NS_ERROR_NOT_AVAILABLE): file /builds/worker/checkouts/gecko/caps/BasePrincipal.cpp:111
[task 2021-04-14T19:50:28.242Z] 19:50:28     INFO - TEST-UNEXPECTED-FAIL | /css/css-ui/text-overflow-022.html | Testing http://web-platform.test:8000/css/css-ui/text-overflow-022.html == http://web-platform.test:8000/css/css-ui/reference/text-overflow-022-ref.html
[task 2021-04-14T19:50:28.242Z] 19:50:28     INFO - Found 12 pixels different, maximum difference per channel 192
[task 2021-04-14T19:50:28.242Z] 19:50:28     INFO - REFTEST   IMAGE 1 (TEST): data:image/png;base64,
[task 2021-04-14T19:50:28.242Z] 19:50:28     INFO - REFTEST   IMAGE 2 (REFERENCE): data:image/png;base64,
[task 2021-04-14T19:50:28.242Z] 19:50:28     INFO - TEST-INFO took 263ms```
Assignee: nobody → bwerth
Flags: needinfo?(bwerth)
Pushed by archaeopteryx@coole-files.de: https://hg.mozilla.org/integration/autoland/rev/7f8d1b667187 disable text-overflow-022.html on Mac for frequent failures. r=aryx DONTBUILD
Attachment #9216616 - Attachment description: Bug 1705193 - disable text-overflow-022.html on Mac for frequent failures. r=aryx → Bug 1705193 - set text-overflow-022.html as failing on Mac debug due to frequent failures. r=aryx
Flags: needinfo?(aryx.bugmail)
Whiteboard: [checkin-needed-beta]
Pushed by archaeopteryx@coole-files.de: https://hg.mozilla.org/integration/autoland/rev/7b7232360d51 set text-overflow-022.html as failing on Mac debug due to frequent failures. r=aryx
Pushed by archaeopteryx@coole-files.de: https://hg.mozilla.org/integration/autoland/rev/3c4ddda8d3e8 set text-overflow-022.html as intermittently failing on Mac debug. DONTBUILD

I couldn't make any sense of this, since the error is in the reference file, and appears to be a truncated ellipsis glyph. So I ran mozregression myself and found Bug 1704255 as the regressor, which matches what I'm seeing. I'm setting that as the regressing bug and taking myself off this bug.

Assignee: bwerth → nobody
Flags: needinfo?(jfkthame)
Regressed by: 1704255
No longer regressed by: 1675375
Has Regression Range: --- → yes

I see what's happening here: the reference file includes the <span>l&#x20DE;</span> after the ellipsis, presumably to ensure it gets the same baseline as the testcase, and sets its color to white so that it won't show up. But ⃞ is an enclosing mark that surrounds the previous character, and depending on the fonts that get used, it can end up slightly overlapping and therefore erasing part of the ellipsis.

A simple fix should be to make this span transparent rather than white, to ensure it is truly invisible and doesn't affect the visible text regardless of exactly where the combining-square glyph ends up.

Flags: needinfo?(jfkthame)
Pushed by jkew@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b38986a96a39 Use 'transparent' rather than 'white' to hide text in the reference file, to avoid it overprinting the ellipsis. r=emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/28642 for changes under testing/web-platform/tests
Keywords: leave-open
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
Assignee: nobody → abutkovits
Upstream PR merged by moz-wptsync-bot
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: