75737 - [FIX]IFrame borders are drawn over when printing

Status: VERIFIED FIXED

(Core :: Printing: Output, defect, P1)

Description

[rods (gone)]

Comment 1

[rods (gone)]

accepting, will attach fix

Comment 2

[rods (gone)]

Temporary fix until 75739 is fixed

Index: nsFrameFrame.cpp
===================================================================
RCS file: /cvsroot/mozilla/layout/html/document/src/nsFrameFrame.cpp,v
retrieving revision 3.143
diff -u -r3.143 nsFrameFrame.cpp
--- nsFrameFrame.cpp    2001/04/12 08:16:53     3.143
+++ nsFrameFrame.cpp    2001/04/12 16:23:00
@@ -692,7 +692,12 @@
   //printf("inner paint %X (%d,%d,%d,%d) \n", this, aDirtyRect.x, aDirtyRect.y,
 aDirtyRect.width, aDirtyRect.height);
   // if there is not web shell paint based on our background color,
   // otherwise let the web shell paint the sub document
-  if (!mSubShell) {
+
+  // isPaginated is a temporary fix for Bug 75737
+  // and this should all be fixed correctly by Bug 75739
+  PRBool isPaginated;
+  aPresContext->IsPaginated(&isPaginated);
+   if (!mSubShell && !isPaginated) {
     const nsStyleColor* color =
       (const nsStyleColor*)mStyleContext->GetStyleData(eStyleStruct_Color);
     aRenderingContext.SetColor(color->mBackgroundColor);

Comment 3

[Marc Attinasi]

Looks like a good temporary fix: sr=[e-mail]

Comment 4

[Eric Pollmann]

Looks good to me too, r=[e-mail].

Comment 5

[rods (gone)]

fixed

Comment 6

[sujay]

Rod, can you veify this if it got fixed? thanks...

Comment 7

[sujay]

verified.