Bug 171802. Make sure views are initialized with the right properties! r=dbaron,sr=kin

layout/base/nsCSSFrameConstructor.cpp

@@ -3626,6 +3626,9 @@ nsCSSFrameConstructor::ConstructRootFrame(nsIPresShell*        aPresShell,
   viewManager->GetRootView(rootView);
   viewportFrame->SetView(aPresContext, rootView);
 
+  nsContainerFrame::SyncFrameViewProperties(aPresContext, viewportFrame,
+                                            viewportPseudoStyle, rootView);
+
   // The viewport is the containing block for 'fixed' elements
   mFixedContainingBlock = viewportFrame;
 

layout/generic/nsFrameFrame.cpp

@@ -1146,6 +1146,9 @@ nsHTMLFrameInnerFrame::CreateViewAndWidget(nsIPresContext* aPresContext,
   rv = view->CreateWidget(kCChildCID, &initData);
   SetView(aPresContext, view);
 
+  nsContainerFrame::SyncFrameViewProperties(aPresContext, this, nsnull, view);
+
+  // XXX the following should be unnecessary, given the above Sync call
   // if the visibility is hidden, reflect that in the view
   const nsStyleVisibility* vis;
   GetStyleData(eStyleStruct_Visibility, ((const nsStyleStruct *&)vis));

layout/generic/nsViewportFrame.cpp

@@ -96,6 +96,8 @@ public:
                     nsHTMLReflowMetrics&     aDesiredSize,
                     const nsHTMLReflowState& aReflowState,
                     nsReflowStatus&          aStatus);
+
+  virtual PRBool CanPaintBackground() { return PR_FALSE; }
   
   /**
    * Get the "type" of the frame

layout/html/base/src/nsViewportFrame.cpp

@@ -96,6 +96,8 @@ public:
                     nsHTMLReflowMetrics&     aDesiredSize,
                     const nsHTMLReflowState& aReflowState,
                     nsReflowStatus&          aStatus);
+
+  virtual PRBool CanPaintBackground() { return PR_FALSE; }
   
   /**
    * Get the "type" of the frame

layout/html/document/src/nsFrameFrame.cpp

@@ -1146,6 +1146,9 @@ nsHTMLFrameInnerFrame::CreateViewAndWidget(nsIPresContext* aPresContext,
   rv = view->CreateWidget(kCChildCID, &initData);
   SetView(aPresContext, view);
 
+  nsContainerFrame::SyncFrameViewProperties(aPresContext, this, nsnull, view);
+
+  // XXX the following should be unnecessary, given the above Sync call
   // if the visibility is hidden, reflect that in the view
   const nsStyleVisibility* vis;
   GetStyleData(eStyleStruct_Visibility, ((const nsStyleStruct *&)vis));

layout/html/style/src/nsCSSFrameConstructor.cpp

@@ -3626,6 +3626,9 @@ nsCSSFrameConstructor::ConstructRootFrame(nsIPresShell*        aPresShell,
   viewManager->GetRootView(rootView);
   viewportFrame->SetView(aPresContext, rootView);
 
+  nsContainerFrame::SyncFrameViewProperties(aPresContext, viewportFrame,
+                                            viewportPseudoStyle, rootView);
+
   // The viewport is the containing block for 'fixed' elements
   mFixedContainingBlock = viewportFrame;