Refactor DrawingRecorders to check for cached drawings earlier

Source/core/paint/TableSectionPainter.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/paint/TableSectionPainter.h"
 
 #include "core/layout/LayoutTable.h"
 #include "core/layout/LayoutTableCell.h"
 #include "core/layout/LayoutTableCol.h"
@@ skipping 158 matching lines @@
         LayoutTableCol* column = m_layoutTableSection.table()->colElement(cell->col());
         LayoutTableCol* columnGroup = column ? column->enclosingColumnGroup() : 0;
 
         bool columnHasBackground = column && column->hasBackground();
         bool columnGroupHasBackground = columnGroup && columnGroup->hasBackground();
         bool sectionHasBackground = m_layoutTableSection.hasBackground();
         bool rowHasBackground = row->hasBackground();
 
         if (columnHasBackground || columnGroupHasBackground || sectionHasBackground || rowHasBackground) {
             TableCellPainter tableCellPainter(*cell);
-            LayoutObjectDrawingRecorder recorder(*paintInfo.context, *cell, paintPhase, tableCellPainter.paintBounds(cellPoint, TableCellPainter::AddOffsetFromParent));
-            if (!recorder.canUseCachedDrawing()) {
+            if (!LayoutObjectDrawingRecorder::useCachedDrawingIfPossible(*paintInfo.context, *cell, paintPhase)) {
+                LayoutObjectDrawingRecorder recorder(*paintInfo.context, *cell, paintPhase, tableCellPainter.paintBounds(cellPoint, TableCellPainter::AddOffsetFromParent));
                 // Column groups and columns first.
                 // FIXME: Columns and column groups do not currently support opacity, and they are being painted "too late" in
                 // the stack, since we have already opened a transparency layer (potentially) for the table row group.
                 // Note that we deliberately ignore whether or not the cell has a layer, since these backgrounds paint "behind" the
                 // cell.
                 if (columnGroupHasBackground)
                     tableCellPainter.paintBackgroundsBehindCell(paintInfo, cellPoint, columnGroup);
                 if (columnHasBackground)
                     tableCellPainter.paintBackgroundsBehindCell(paintInfo, cellPoint, column);
 
                 // Paint the row group next.
                 if (sectionHasBackground)
                     tableCellPainter.paintBackgroundsBehindCell(paintInfo, cellPoint, &m_layoutTableSection);
 
                 // Paint the row next, but only if it doesn't have a layer. If a row has a layer, it will be responsible for
                 // painting the row background for the cell.
                 if (rowHasBackground && !row->hasSelfPaintingLayer())
                     tableCellPainter.paintBackgroundsBehindCell(paintInfo, cellPoint, row);
             }
         }
     }
     if ((!cell->hasSelfPaintingLayer() && !row->hasSelfPaintingLayer()))
         cell->paint(paintInfo, cellPoint);
 }
 
 } // namespace blink