Now that we have a more standard conform way to control if margin boxes should be printed or not via CSS (by setting margin to 0 in a @page block), as implemented in https://bugzilla.mozilla.org/show_bug.cgi?id=1250674, I suggest removing the unofficial and redundant mozNoMarginBoxes.
Comment on attachment 8735925 [details] [diff] [review] Remove mozNoMarginBoxes David, asking you to review since you also reviewed https://bugzilla.mozilla.org/show_bug.cgi?id=1250674, which aimed to replace this feature by a more standard conform one. I already talked to the PDF.js team, which originally implemented this feature, they don't have any concerns.
Comment on attachment 8735925 [details] [diff] [review] Remove mozNoMarginBoxes Please also remove persistMarginBoxSettings from: https://mxr.mozilla.org/mozilla-central/source/embedding/components/printingui/ipc/PPrintingTypes.ipdlh#71 r=dbaron with that Thanks for cleaing this up.
Created attachment 8738618 [details] [diff] [review] Remove mozNoMarginBoxes (v2) Addressed review comment.