Closed Bug 1908583 Opened 2 years ago Closed 1 year ago

Intermittent selection/1478604.html == selection/1478604-ref.html | single tracking bug

Categories

(Core :: DOM: Selection, defect, P5)

defect

Tracking

()

RESOLVED INCOMPLETE

People

(Reporter: intermittent-bug-filer, Unassigned)

References

Details

(Keywords: intermittent-failure, intermittent-testcase)

Attachments

(3 files)

The testcase screenshot here shows two copies of the test content for some reason, and the lower copy is the unexpected part.

This might be related to whatever's going wrong in bug 1912571, since they tests are in the same directory and are failing on the same android-em-7-0-x86_64-qr platform, and I observed some bizarre/unexpected content in the rendering in bug 1912571 comment 11.

I wonder if we've got some sort of general invalidation/stale-painting bug on this platform?

(In reply to Daniel Holbert [:dholbert] from comment #28)

The testcase screenshot here shows two copies of the test content for some reason, and the lower copy is the unexpected part.

Illustrating this -- here's the testcase screenshot from this recent failure log:
https://treeherder.mozilla.org/logviewer?job_id=490831366&repo=autoland&lineNumber=8335

The reference screenshot looks the same, except that it doesn't have the unexpected/duplicated content in the bottom half of its screenshot.

It's not just the testcase misrendering here; sometimes the reference case misrenders instead. I've got two examples from a recent try run:
(1) https://treeherder.mozilla.org/logviewer?job_id=491843028&repo=try&lineNumber=1796
Here, the testcase looks correct, and the reference case has the duplicated content as shown in comment 29.

(2) https://treeherder.mozilla.org/logviewer?job_id=491843037&repo=try&lineNumber=1793
Here, the testcase looks correct, and the reference case looks similar to comment 29, except with the lower duplicated/unexpected-portion clipped on its bottom edge and shifted rightwards.

See Also: → 1912571

This patch replaces the stale code-comment with an up-to-date one for the skip
annotation that I'm adding here.

The stale comment was documenting a fuzzy annotation that was removed long ago,
which was specific to non-webrender builds (which aren't possible / supported
at this point).

Assignee: nobody → dholbert
Status: NEW → ASSIGNED

This test seems to be in the same rough situation as bug 1912571, so let's skip it on Android-emulator, to stop the onslaught[1] of test failures for now.

[1] In a Try run that I pushed last night (as part of testing my annotation for bug 1912571), this specific test failed in 30% (9 out of 30) of the retriggers:
https://treeherder.mozilla.org/jobs?repo=try&revision=039212df18941b033e7495d9cb6fa1c9fd6be791

Assignee: dholbert → nobody
Status: ASSIGNED → NEW
Keywords: leave-open
Pushed by dholbert@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/802dcc7fb6d5 Skip reftest selection/1478604.html on Android emulator for frequent failures. r=hiro

Note, a few of these have a single-random-pixel mismatch:
This one's got a single pixel about halfway down on the left side that's light gray in the reference case screenshot (when it should be white):
https://treeherder.mozilla.org/logviewer?job_id=492249213&repo=autoland&lineNumber=8291

This one looks similar except that it's the testcase with the gray pixel, and the gray pixel is near the center of the screenshot:
https://treeherder.mozilla.org/logviewer?job_id=492249215&repo=autoland&lineNumber=8292

This patch replaces the stale code-comment with an up-to-date one for the skip
annotation that I'm adding here.

The stale comment was documenting a fuzzy annotation that was removed long ago,
which was specific to non-webrender builds (which aren't possible / supported
at this point).

Original Revision: https://phabricator.services.mozilla.com/D236725

Attachment #9465765 - Flags: approval-mozilla-beta?

beta Uplift Approval Request

  • User impact if declined: None
  • Code covered by automated testing: yes
  • Fix verified in Nightly: yes
  • Needs manual QE test: no
  • Steps to reproduce for manual QE testing: N/A
  • Risk associated with taking this patch: None
  • Explanation of risk level: Just adding a skip-on-android-emulator annotation for some intermittently-failing reftests (which we're now skipping on central but are still failing on beta)
  • String changes made/needed: None
  • Is Android affected?: yes
Attachment #9465765 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: