Building with --disable-printing is broken
Categories
(Core :: Printing: Setup, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | unaffected |
firefox-esr78 | --- | unaffected |
firefox80 | --- | unaffected |
firefox81 | --- | fixed |
firefox82 | --- | fixed |
People
(Reporter: fabrice, Assigned: emilio)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
Regressed by https://bugzilla.mozilla.org/show_bug.cgi?id=1661388 changes to nsGlobalWindowOuter.cpp
Updated•4 years ago
|
Comment 1•4 years ago
|
||
Set release status flags based on info from the regressing bug 1661388
Assignee | ||
Comment 2•4 years ago
|
||
I can take this. What are you using --disable-printing
for?
Reporter | ||
Comment 4•4 years ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #2)
I can take this. What are you using
--disable-printing
for?
b2g builds for KaiOS.
Assignee | ||
Comment 5•4 years ago
|
||
(In reply to [:fabrice] Fabrice Desré from comment #4)
b2g builds for KaiOS.
Does it have a big enough code-size impact that it warrants disabling? Android just disables window.print at runtime, and stubs the print service.
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 6•4 years ago
|
||
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0f3afede4904 Fix builds with --disable-printing. r=jwatt
Comment 8•4 years ago
|
||
bugherder |
Comment 9•4 years ago
|
||
Comment on attachment 9173677 [details]
Bug 1661838 - Fix builds with --disable-printing. r=#layout-reviewers
Beta/Release Uplift Approval Request
- User impact if declined: --disable-printing being broken for builds other people create. maybe also small chance of conflict with other patches we need.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky):
- String changes made/needed:
Comment 10•4 years ago
|
||
Comment on attachment 9173677 [details]
Bug 1661838 - Fix builds with --disable-printing. r=#layout-reviewers
Approved for 81.0b6.
Comment 11•4 years ago
|
||
bugherder uplift |
Updated•4 years ago
|
Reporter | ||
Comment 12•4 years ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #5)
(In reply to [:fabrice] Fabrice Desré from comment #4)
b2g builds for KaiOS.
Does it have a big enough code-size impact that it warrants disabling? Android just disables window.print at runtime, and stubs the print service.
I don't know the code size impact, but I would say that if there's a build time flag it should work :) Thanks for fixing it anyway!
Updated•4 years ago
|
Description
•