The SavePrintSettingsToPrefs call in nsPrintDialogServiceGTK::ShowPageSetup shouldn't save all settings
Categories
(Core :: Printing: Setup, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox78 | --- | fixed |
People
(Reporter: jwatt, Assigned: hiro)
References
Details
(Whiteboard: [print2020_v78])
Attachments
(2 files)
This SavePrintSettingsToPrefs call was added when ventnor took over the work and added his first patch in bug 193001 comment 42. There's no explanation there as to why he added the call or specifically why the GTK code is special.
I'd note that the mac code essentially does an extra save in the same place (ShowPageSetup - see bug 1636668), but with kInitSaveNativeData rather than InitSaveAll.
Reporter | ||
Comment 1•4 years ago
|
||
(It might be helpful to note that kInitSaveNativeData isn't handled explicitly. That constant is essentially mac only because the mac code is the only platform to override nsPrintSettingsService::WritePrefs.)
Reporter | ||
Comment 2•4 years ago
|
||
(In reply to Jonathan Watt [:jwatt] from comment #0)
I'd note that the mac code essentially does an extra save in the same place (ShowPageSetup - see bug 1636668), but with kInitSaveNativeData rather than InitSaveAll.
Turns out the Mac code is broken and does need to save more of the settings, but that was for the most part being hidden. (See the bug given above for details.)
So this code does appear to be necessary, but we should only really be saving the settings that the Page Setup dialog actually shows.
Hiro, do you want to take this?
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 4•4 years ago
|
||
Assignee | ||
Comment 5•4 years ago
|
||
That's all properties GtkPageSetup has [1].
[1] https://developer.gnome.org/gtk3/stable/GtkPageSetup.html
Depends on D75898
Comment 7•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ef5347cdad10
https://hg.mozilla.org/mozilla-central/rev/8fdc6cd9df8b
Description
•