Address review comment on bug 9458 that depends on bug 367247 by making nsFrame::IsContainingBlock not check display type, but making it purely frame-class-based.

layout/generic/nsFrame.cpp

@@ -3766,17 +3766,9 @@ nsFrame::IsFrameTreeTooDeep(const nsHTMLReflowState& aReflowState,
   return PR_FALSE;
 }
 
-// Style sizing methods
 /* virtual */ PRBool nsFrame::IsContainingBlock() const
 {
-  const nsStyleDisplay* display = GetStyleDisplay();
-
-  // Absolute positioning causes |display->mDisplay| to be set to block,
-  // if needed.
-  return display->mDisplay == NS_STYLE_DISPLAY_BLOCK || 
-         display->mDisplay == NS_STYLE_DISPLAY_INLINE_BLOCK || 
-         display->mDisplay == NS_STYLE_DISPLAY_LIST_ITEM ||
-         display->mDisplay == NS_STYLE_DISPLAY_TABLE_CELL;
+  return PR_FALSE;
 }
 
 #ifdef NS_DEBUG

layout/tables/nsTableCellFrame.cpp

@@ -964,6 +964,12 @@ nsTableCellFrame::GetType() const
   return nsGkAtoms::tableCellFrame;
 }
 
+/* virtual */ PRBool
+nsTableCellFrame::IsContainingBlock() const
+{
+  return PR_TRUE;
+}
+
 #ifdef DEBUG
 NS_IMETHODIMP
 nsTableCellFrame::GetFrameName(nsAString& aResult) const

layout/tables/nsTableCellFrame.h

@@ -148,6 +148,8 @@ public:
    */
   virtual nsIAtom* GetType() const;
 
+  virtual PRBool IsContainingBlock() const;
+
 #ifdef DEBUG
   NS_IMETHOD GetFrameName(nsAString& aResult) const;
 #endif