Paint backgrounds of a table section/row in one display item

third_party/WebKit/Source/core/layout/LayoutTableCell.cpp

 /*
  * Copyright (C) 1997 Martin Jones (mjones@kde.org)
  *           (C) 1997 Torben Weis (weis@kde.org)
  *           (C) 1998 Waldo Bastian (bastian@kde.org)
  *           (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc.
  *               All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 30 matching lines @@
 #include "wtf/PtrUtil.h"
 
 namespace blink {
 
 using namespace HTMLNames;
 
 struct SameSizeAsLayoutTableCell : public LayoutBlockFlow {
   unsigned bitfields;
   int paddings[2];
   void* pointer1;
-  void* pointer2;
 };
 
 static_assert(sizeof(LayoutTableCell) == sizeof(SameSizeAsLayoutTableCell),
               "LayoutTableCell should stay small");
 static_assert(sizeof(CollapsedBorderValue) == 8,
               "CollapsedBorderValue should stay small");
 
 LayoutTableCell::LayoutTableCell(Element* element)
     : LayoutBlockFlow(element),
       m_absoluteColumnIndex(unsetColumnIndex),
@@ skipping 398 matching lines @@
   // The baseline of a cell is the baseline of the first in-flow line box in the
   // cell, or the first in-flow table-row in the cell, whichever comes first. If
   // there is no such line box or table-row, the baseline is the bottom of
   // content edge of the cell box.
   int firstLineBaseline = firstLineBoxBaseline();
   if (firstLineBaseline != -1)
     return firstLineBaseline;
   return (borderBefore() + paddingBefore() + contentLogicalHeight()).toInt();
 }
 
-void LayoutTableCell::ensureIsReadyForPaintInvalidation() {
-  LayoutBlockFlow::ensureIsReadyForPaintInvalidation();
-  if (!usesCompositedCellDisplayItemClients())
-    return;
-  if (!m_rowBackgroundDisplayItemClient) {
-    m_rowBackgroundDisplayItemClient = WTF::wrapUnique(
-        new LayoutTableCell::RowBackgroundDisplayItemClient(*this));
-  }
-}
-
 void LayoutTableCell::styleDidChange(StyleDifference diff,
                                      const ComputedStyle* oldStyle) {
   DCHECK_EQ(style()->display(), EDisplay::kTableCell);
 
   LayoutBlockFlow::styleDidChange(diff, oldStyle);
   setHasBoxDecorationBackground(true);
 
   if (parent() && section() && oldStyle &&
       style()->height() != oldStyle->height())
     section()->rowLogicalHeightChanged(row());
@@ skipping 25 matching lines @@
     }
     if (nextCell()) {
       // TODO(dgrogan) Add a layout test showing that setChildNeedsLayout is
       // needed instead of setNeedsLayout.
       nextCell()->setChildNeedsLayout();
       nextCell()->setPreferredLogicalWidthsDirty(MarkOnlyThis);
     }
   }
 }
 
-LayoutTableCell::RowBackgroundDisplayItemClient::RowBackgroundDisplayItemClient(
-    const LayoutTableCell& layoutTableCell)
-    : m_layoutTableCell(layoutTableCell) {}
-
-String LayoutTableCell::RowBackgroundDisplayItemClient::debugName() const {
-  return "RowBackground";
-}
-
-LayoutRect LayoutTableCell::RowBackgroundDisplayItemClient::visualRect() const {
-  return m_layoutTableCell.row()->visualRect();
-}
-
 // The following rules apply for resolving conflicts and figuring out which
 // border to use.
 // (1) Borders with the 'border-style' of 'hidden' take precedence over all
 //     other conflicting borders. Any border with this value suppresses all
 //     borders at this location.
 // (2) Borders with a style of 'none' have the lowest priority. Only if the
 //     border properties of all the elements meeting at this edge are 'none'
 //     will the border be omitted (but note that 'none' is the default value for
 //     the border style.)
 // (3) If none of the styles are 'hidden' and at least one of them is not
@@ skipping 917 matching lines @@
 void LayoutTableCell::invalidateDisplayItemClients(
     PaintInvalidationReason reason) const {
   LayoutBlockFlow::invalidateDisplayItemClients(reason);
 
   if (!usesCompositedCellDisplayItemClients())
     return;
 
   ObjectPaintInvalidator invalidator(*this);
   if (m_collapsedBorderValues)
     invalidator.invalidateDisplayItemClient(*m_collapsedBorderValues, reason);
-  if (m_rowBackgroundDisplayItemClient) {
-    invalidator.invalidateDisplayItemClient(*m_rowBackgroundDisplayItemClient,
-                                            reason);
-  }
 }
 
 // TODO(lunalu): Deliberately dump the "inner" box of table cells, since that
 // is what current results reflect.  We'd like to clean up the results to dump
 // both the outer box and the intrinsic padding so that both bits of information
 // are captured by the results.
 LayoutRect LayoutTableCell::debugRect() const {
   LayoutRect rect = LayoutRect(
       location().x(), location().y() + intrinsicPaddingBefore(), size().width(),
       size().height() - intrinsicPaddingBefore() - intrinsicPaddingAfter());
@@ skipping 10 matching lines @@
 }
 
 bool LayoutTableCell::hasLineIfEmpty() const {
   if (node() && hasEditableStyle(*node()))
     return true;
 
   return LayoutBlock::hasLineIfEmpty();
 }
 
 }  // namespace blink