superreview granted: [Bug 294717] crash in [@ nsFieldSetFrame::SetInitialChildList] printing <fieldset><legend> : [Attachment 187320] Patch rev. 1
David Baron <dbaron@mozillafoundation.org> has granted Mats Palmgren <mats.palmgren@bredband.net>'s request for superreview: Bug 294717: crash in [@ nsFieldSetFrame::SetInitialChildList] printing <fieldset><legend> https://bugzilla.mozilla.org/show_bug.cgi?id=294717 Attachment 187320: Patch rev. 1 https://bugzilla.mozilla.org/attachment.cgi?id=187320&action=edit ...

superreview granted: [Bug 303620] Focus rectangle drawn at <select> when focus is actually on <input> : [Attachment 192375] Very sensible change -- same exact thing for button, select, input and text
Brian Ryner <bryner@brianryner.com> has granted Aaron Leventhal <aaronleventhal@moonset.net>'s request for superreview: Bug 303620: Focus rectangle drawn at <select> when focus is actually on <input> https://bugzilla.mozilla.org/show_bug.cgi?id=303620 Attachment 192375: Very sensible change -- same exact thing for button, select, input and textarea. Don't assume that focus stays what you set it to after a SetContentState(NS_EVENT_STATE_FOCUS) https://bugzilla.mozilla.org/attachment.cgi?id=192375&action=edit ...

superreview granted: [Bug 232545] Text included by <marquee></marquee> is not shown : [Attachment 140495] Proposed patch
Boris Zbarsky <bz-vacation@mit.edu> has granted Boris Zbarsky <bz-vacation@mit.edu>'s request for superreview: Bug 232545: Text included by <marquee></marquee> is not shown http://bugzilla.mozilla.org/show_bug.cgi?id=232545 Attachment 140495: Proposed patch http://bugzilla.mozilla.org/attachment.cgi?id=140495&action=edit ------- Additional Comments from Boris Zbarsky <bz-vacation@mit.edu> sr=bzbarsky, and please try to lxr for patterns that may have been broken by the other patch, ok? ...

superreview granted: [Bug 126802] Printing while in print preview disabled on Unix : [Attachment 333675] patch to enable printing while in print preview
Robert O'Callahan (:roc) (Mozilla Corporation) <roc@ocallahan.org> has granted Leon Sha <leon.sha@sun.com>'s request for superreview: Bug 126802: Printing while in print preview disabled on Unix https://bugzilla.mozilla.org/show_bug.cgi?id=126802 Attachment 333675: patch to enable printing while in print preview https://bugzilla.mozilla.org/attachment.cgi?id=333675&action=edit ...

superreview granted: [Bug 226871] text in <tt> .. </tt> uses proportional font : [Attachment 136650] patch, with a lot of cleanup
Boris Zbarsky (not really reading bugmail) <bz-vacation@mit.edu> has granted David Baron <dbaron@dbaron.org>'s request for superreview: Bug 226871: text in <tt> .. </tt> uses proportional font http://bugzilla.mozilla.org/show_bug.cgi?id=226871 Attachment 136650: patch, with a lot of cleanup http://bugzilla.mozilla.org/attachment.cgi?id=136650&action=edit ------- Additional Comments from Boris Zbarsky (not really reading bugmail) <bz-vacation@mit.edu> Looks good. r+sr=bzbarsky ...

superreview requested: [Bug 294717] crash in [@ nsFieldSetFrame::SetInitialChildList] printing <fieldset><legend> : [Attachment 187320] Patch rev. 1
Mats Palmgren <mats.palmgren@bredband.net> has asked David Baron <dbaron@mozillafoundation.org> for superreview: Bug 294717: crash in [@ nsFieldSetFrame::SetInitialChildList] printing <fieldset><legend> https://bugzilla.mozilla.org/show_bug.cgi?id=294717 Attachment 187320: Patch rev. 1 https://bugzilla.mozilla.org/attachment.cgi?id=187320&action=edit ------- Additional Comments from Mats Palmgren <mats.palmgren@bredband.net> Fixes the crash (last hunk, in |CreateContinuingFrame|), also fixes |GetAbsoluteContainingBlock| which would return the lege...

superreview granted: [Bug 204615] Editor hangs when pasting a list fragment <LI> in <DL>, <UL> or <OL> : [Attachment 171288] patch v1
Daniel Veditz <dveditz@cruzio.com> has granted Daniel Veditz <dveditz@cruzio.com>'s request for superreview: Bug 204615: Editor hangs when pasting a list fragment <LI> in <DL>, <UL> or <OL> https://bugzilla.mozilla.org/show_bug.cgi?id=204615 Attachment 171288: patch v1 https://bugzilla.mozilla.org/attachment.cgi?id=171288&action=edit ------- Additional Comments from Daniel Veditz <dveditz@cruzio.com> sr=dveditz. Really should have an editor peer review, but I guess this is trivial enough. ...

superreview granted: [Bug 244569] Incorrect font style after <p></font> inside <table> inside <font> : [Attachment 204389] Potential patch
Johnny Stenback <jst@mozilla.org> has granted Johnny Stenback <jst@mozilla.org>'s request for superreview: Bug 244569: Incorrect font style after <p></font> inside <table> inside <font> https://bugzilla.mozilla.org/show_bug.cgi?id=244569 Attachment 204389: Potential patch https://bugzilla.mozilla.org/attachment.cgi?id=204389&action=edit ...

superreview granted: [Bug 223737] {inc}display several extra newlines for <img>text<br clear="all"> : [Attachment 185937] patch
Robert O'Callahan (Novell) <roc@ocallahan.org> has granted Hideo Saito <saito@densan.co.jp>'s request for superreview: Bug 223737: {inc}display several extra newlines for <img>text<br clear="all"> https://bugzilla.mozilla.org/show_bug.cgi?id=223737 Attachment 185937: patch https://bugzilla.mozilla.org/attachment.cgi?id=185937&action=edit ...

superreview requested: [Bug 298766] the lay-out of the help item about print preview is weird : [Attachment 187282] reformats the list, adding moving a few </ul> and </li>s
Steffen Wilberg <steffen.wilberg@web.de> has asked for superreview: Bug 298766: the lay-out of the help item about print preview is weird https://bugzilla.mozilla.org/show_bug.cgi?id=298766 Attachment 187282: reformats the list, adding moving a few </ul> and </li>s https://bugzilla.mozilla.org/attachment.cgi?id=187282&action=edit ------- Additional Comments from Steffen Wilberg <steffen.wilberg@web.de> r=me, indenting needs to be fixed before checkin. ...

superreview requested: [Bug 223737] {inc}display several extra newlines for <img>text<br clear="all"> : [Attachment 190229] patch for skip to reflow lines of a text control frame
Hideo Saito <saito@densan.co.jp> has asked Robert O'Callahan (Novell) <roc@ocallahan.org> for superreview: Bug 223737: {inc}display several extra newlines for <img>text<br clear="all"> https://bugzilla.mozilla.org/show_bug.cgi?id=223737 Attachment 190229: patch for skip to reflow lines of a text control frame https://bugzilla.mozilla.org/attachment.cgi?id=190229&action=edit ------- Additional Comments from Hideo Saito <saito@densan.co.jp> In order to skip to reflow lines of a text control frame, a bit of NS_FRAME_INDEPENDENT_SELECTION...

superreview denied: [Bug 223737] {inc}display several extra newlines for <img>text<br clear="all"> : [Attachment 190229] patch for skip to reflow lines of a text control frame
Robert O'Callahan (Novell) <roc@ocallahan.org> has denied Hideo Saito <saito@densan.co.jp>'s request for superreview: Bug 223737: {inc}display several extra newlines for <img>text<br clear="all"> https://bugzilla.mozilla.org/show_bug.cgi?id=223737 Attachment 190229: patch for skip to reflow lines of a text control frame https://bugzilla.mozilla.org/attachment.cgi?id=190229&action=edit ...

superreview granted: [Bug 293224] [FIX]Matrix from getScreenCTM should be to initial viewport space : [Attachment 194961] patch
T Rowley (IBM) <tor@acm.org> has granted Jonathan Watt <jonathan.watt@strath.ac.uk>'s request for superreview: Bug 293224: [FIX]Matrix from getScreenCTM should be to initial viewport space https://bugzilla.mozilla.org/show_bug.cgi?id=293224 Attachment 194961: patch - include currentScale and Translate for <svg><svg></svg></svg> https://bugzilla.mozilla.org/attachment.cgi?id=194961&action=edit ...

superreview granted: [Bug 172525] [ps] print legal size page fails to set '<< /PageSize [612 1008] >> setpagedevice' : [Attachment 135001] Set PageSize and ImagingBBox
<tor@acm.org> has granted timeless@myrealbox.com <timeless@bemail.org>'s request for superreview: Bug 172525: [ps] print legal size page fails to set '<< /PageSize [612 1008] >> setpagedevice' http://bugzilla.mozilla.org/show_bug.cgi?id=172525 Attachment 135001: Set PageSize and ImagingBBox http://bugzilla.mozilla.org/attachment.cgi?id=135001&action=edit ...