1984884 - After update 142.0, problem with printing the selection menu in html "select name" only an empty frame is displayed.

Status: VERIFIED FIXED

(Core :: Printing: Output, defect)

Description

[Tomas]

Attachment: Screenshot 2025-08-24 09.png

Steps to reproduce:

All browsers with new update. After update 142.0, the print request cannot be printed from the page where the code option is selected.

<select name="platba"><option value="jb">Hotovost</option><option selected="selected" value="hs">Platební kartou</option></select>
<a name="option"></a>

Actual results:

only an empty frame is displayed without the selected item

Expected results:

print the selected value

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Core::Printing: Output' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 2

[Jonathan Kew [:jfkthame]]

Mozregression says this is a result of bug 1873301.

Marking S2 as it's a recent regression, and causes some content to unexpectedly go missing in print/pdf output.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1873301

Comment 4

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1984884 - Set displayed index in nsComboboxControlFrame::Init. r=#layout-reviewers

So that we have the right index when creating the anonymous content,
without needing to reflow first.

This is the low-risk fix for uplift. More complete fix to come.

Comment 5

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1984884 - Stop re-setting the selected text in nsComboboxControlFrame::Reflow. r=#layout

This is what papered over this in non-printing scenarios.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/9c440f3fb4cf
https://hg.mozilla.org/integration/autoland/rev/fa5150f4a96a
Set displayed index in nsComboboxControlFrame::Init. r=layout-reviewers,tnikkel

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/54497 for changes under testing/web-platform/tests

Comment 8

[Emilio Cobos Álvarez [:emilio]]

Comment on attachment 9509144 [details]
Bug 1984884 - Set displayed index in nsComboboxControlFrame::Init. r=#layout-reviewers

Beta/Release Uplift Approval Request

• User impact if declined/Reason for urgency: missing content in print
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: comment 0
• List of other uplifts needed: none
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Trivial one-liner
• String changes made/needed: none
• Is Android affected?: Yes

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/b4e82f5c1523
https://hg.mozilla.org/integration/autoland/rev/498fd968d39d
Stop re-setting the selected text in nsComboboxControlFrame::Reflow. r=layout-reviewers,dshin

Comment 12

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/fa5150f4a96a
https://hg.mozilla.org/mozilla-central/rev/498fd968d39d

Comment 13

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot

Comment 14

[Dianna Smith [:diannaS]]

Comment on attachment 9509144 [details]
Bug 1984884 - Set displayed index in nsComboboxControlFrame::Init. r=#layout-reviewers

Approved for 143.0b5

Comment 15

[Pulsebot]

https://github.com/mozilla-firefox/firefox/commit/240dddd86800
https://hg.mozilla.org/releases/mozilla-beta/rev/beba693aea55

Comment 17

[Dianna Smith [:diannaS]]

Comment on attachment 9509144 [details]
Bug 1984884 - Set displayed index in nsComboboxControlFrame::Init. r=#layout-reviewers

Approved for 142.0.1

Comment 18

[Pulsebot]

https://github.com/mozilla-firefox/firefox/commit/88d8de5e45ac
https://hg.mozilla.org/releases/mozilla-release/rev/1bad1ec158f4

Comment 20

[Bogdan Maris, Desktop Test Engineering [Away till 13th July]]

Reproduced the initial issue using an old Nightly build from 2 days ago. Verified that using latest Nightly 144.0a1, Firefox 142.0.1 and latest Beta 143 build from treeherder across platforms (Windows 10, macOS 13 and Ubuntu 22.04) this is no longer an issue for both print preview or printing to pdf.

Comment 22

[Alex Finder]

(In reply to Cosmin Sabou [:CosminS] from comment #12)

https://hg.mozilla.org/mozilla-central/rev/fa5150f4a96a
https://hg.mozilla.org/mozilla-central/rev/498fd968d39d

Hi Emilio,

Would you agree that revision fa5150f4a96a5160bb0813d921d5ef4c50067f53 caused the improvements mentioned below ?

That revision is in a busted range and we cannot run the android booking jobs on that revision, but based on the initial regression ((bug 1977839)[https://bugzilla.mozilla.org/show_bug.cgi?id=1977839]), this seemed to be the most likely culprit.

Perfherder has detected a browsertime performance change from push fa5150f4a96a5160bb0813d921d5ef4c50067f53.

If you have any questions, please reach out to a performance sheriff. Alternatively, you can find help on Slack by joining #perf-help, and on Matrix you can find help by joining #perftest.

Improvements:

Ratio	Test	Platform	Options	Absolute values (old vs new)
20%	booking largestContentfulPaint	android-hw-a55-14-0-aarch64-shippable	warm webrender	206.97 -> 166.28
19%	booking FirstVisualChange	android-hw-a55-14-0-aarch64-shippable	warm webrender	203.93 -> 164.35
15%	booking fcp	android-hw-a55-14-0-aarch64-shippable	warm webrender	176.89 -> 150.49
9%	booking ContentfulSpeedIndex	android-hw-a55-14-0-aarch64-shippable	warm webrender	291.52 -> 264.78
5%	booking SpeedIndex	android-hw-a55-14-0-aarch64-shippable	warm webrender	530.04 -> 501.57
3%	booking PerceptualSpeedIndex	android-hw-a55-14-0-aarch64-shippable	warm webrender	690.28 -> 669.67

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a performance sheriff to do that for you.

You can run all of these tests on try with ./mach try perf --alert 46515

The following documentation link provides more information about this command.

Comment 23

[Emilio Cobos Álvarez [:emilio]]

Yes that's why I marked https://bugzilla.mozilla.org/show_bug.cgi?id=1977839 as a dupe of this one.