1895896 - Perma /css/css-content/parsing/content-computed.html | Property content value 'open-quote / "alt text"' - assert_true: 'open-quote / "alt text"' is a supported value for content. expected true got false when Gecko 127 merges to beta on 2024-05-13

Status: VERIFIED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Aron Cseh]

Central-as-beta simulation
How to run these simulations
Failure log:

[task 2024-05-09T12:20:25.486Z] 12:20:25     INFO - TEST-PASS | /css/css-content/parsing/content-computed.html | Property content value 'open-quote' 
[task 2024-05-09T12:20:25.486Z] 12:20:25     INFO - TEST-UNEXPECTED-FAIL | /css/css-content/parsing/content-computed.html | Property content value 'open-quote / "alt text"' - assert_true: 'open-quote / "alt text"' is a supported value for content. expected true got false
[task 2024-05-09T12:20:25.486Z] 12:20:25     INFO - test_computed_value/<@http://web-platform.test:8000/css/support/computed-testcommon.js:25:16
[task 2024-05-09T12:20:25.486Z] 12:20:25     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2622:25
[task 2024-05-09T12:20:25.486Z] 12:20:25     INFO - test@http://web-platform.test:8000/resources/testharness.js:633:30
[task 2024-05-09T12:20:25.486Z] 12:20:25     INFO - test_computed_value@http://web-platform.test:8000/css/support/computed-testcommon.js:22:7
[task 2024-05-09T12:20:25.486Z] 12:20:25     INFO - test_computed_value_combinations@http://web-platform.test:8000/css/css-content/parsing/content-computed.html:21:24
[task 2024-05-09T12:20:25.486Z] 12:20:25     INFO - @http://web-platform.test:8000/css/css-content/parsing/content-computed.html:24:33
[task 2024-05-09T12:20:25.487Z] 12:20:25     INFO - 
[task 2024-05-09T12:20:25.487Z] 12:20:25     INFO - TEST-PASS | /css/css-content/parsing/content-computed.html | Property content value 'close-quote' 

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Wrong Emilio?

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5318a409a162
Turn on content alt text pref on css-content tests.

Comment 3

[Iulian Moraru]

The patch was tested in today's central as early beta simulation and it seems to be working.

Comment 4

[Daniel Holbert [:dholbert]]

(Odd; I tried to reclassify this as a task since this is just a test manifest tweak with no user-facing impact, but Bugzilla seems to reject that change, and confusingly shows me as having made the opposite change. I'm going to try one more time, for the benefit of a screencast that I can attach to a bug report that I'll submit for b.m.o, and then I'll stop poking here; sorry for the noise. :) ) [edit: filed this as bug 1896204]

Comment 5

[Donal Meehan [:dmeehan]]

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

(Odd; I tried to reclassify this as a task since this is just a test manifest tweak with no user-facing impact, but Bugzilla seems to reject that change, and confusingly shows me as having made the opposite change. I'm going to try one more time, for the benefit of a screencast that I can attach to a bug report that I'll submit for b.m.o, and then I'll stop poking here; sorry for the noise. :) )

It's flagged as a regression, bugzilla won't allow regressions to be tasks.
The only way to convert to a task is to clear the regressor and the regression keyword (though that would lose tracking so it's not recommended)

Comment 6

[Daniel Holbert [:dholbert]]

(In reply to Donal Meehan [:dmeehan] from comment #5)

It's flagged as a regression, bugzilla won't allow regressions to be tasks.

Thanks - I suspected that might be it. The way-it-doesn't-allow-this could definitely be made less perplexing; hence, filed bug 1896204. :)

The only way to convert to a task is to clear the regressor and the regression keyword (though that would lose tracking so it's not recommended)

Yeah. I'm not sure what the right solution is, but ideally we'd have some way to let bugdash not-include bugs like this in its lists of "real" user-facing regressions, to avoid adding noise to what are otherwise high-signal bug lists. (I happened to notice this bug since it was (for the moment) on bugdash under "127 (Nightly) New Regressions" in the "Important" tab. Definitely worth getting this addressed [and we did, hooray!] to avoid perma-orange when 127 hits beta -- but it feels like it's in a different category from "real" regressions where users are potentially hitting site breakage.)

Comment 7

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/5318a409a162

Comment 8

[Intermittent Failures Robot]

2 failures in 3649 pushes (0.001 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• try: 2

Platform and build breakdown:

• macosx1015-64-shippable-qr: 1
  • opt: 1
• android-em-7-0-x86_64-shippable-qr: 1
  • opt: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1895896&startday=2024-05-06&endday=2024-05-12&tree=all