Revert of Revert of Temporarily let DocumentMakerController use inval…

…idatePaintForWholeRenderer (patchset #2 id:70001 of https://codereview.chromium.org/574773002/)

Reason for revert:
The RenderText change needs to be reverted because it causes failure of some chromium browsertest. 

Original issue's description:
> Revert of Temporarily let DocumentMakerController use invalidatePaintForWholeRenderer (patchset #3 id:60001 of https://codereview.chromium.org/499493003/)
> 
> Reason for revert:
> We have fixed the RenderText invalidation issue (crbug.com/394133).
> 
> Original issue's description:
> > Temporarily let DocumentMakerController use invalidatePaintForWholeRenderer
> > 
> > setShouldDoFullPaintInvalidation doesn't work for RenderText yet.
> > 
> > BUG=405501
> > 
> > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180877
> 
> [email protected],[email protected],[email protected]
> BUG=405501
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=182089

[email protected],[email protected],[email protected]
NOTREECHECKS=true
NOTRY=true
BUG=405501

Review URL: https://codereview.chromium.org/574913002

git-svn-id: svn://svn.chromium.org/blink/trunk@182126 bbb929c8-8fbe-4397-9dbb-9b2b20218538

Source/core/dom/DocumentMarkerController.cpp

@@ -235,7 +235,7 @@ void DocumentMarkerController::addMarker(Node* node, const DocumentMarker& newMa
 
     // repaint the affected node
     if (node->renderer())
-        node->renderer()->setShouldDoFullPaintInvalidation(true);
+        node->renderer()->doNotUseInvalidatePaintForWholeRendererSynchronously();
 }
 
 void DocumentMarkerController::mergeOverlapping(MarkerList* list, DocumentMarker& toInsert)
@@ -296,7 +296,7 @@ void DocumentMarkerController::copyMarkers(Node* srcNode, unsigned startOffset,
 
     // repaint the affected node
     if (docDirty && dstNode->renderer())
-        dstNode->renderer()->setShouldDoFullPaintInvalidation(true);
+        dstNode->renderer()->doNotUseInvalidatePaintForWholeRendererSynchronously();
 }
 
 void DocumentMarkerController::removeMarkers(Node* node, unsigned startOffset, int length, DocumentMarker::MarkerTypes markerTypes, RemovePartiallyOverlappingMarkerOrNot shouldRemovePartiallyOverlappingMarker)
@@ -377,7 +377,7 @@ void DocumentMarkerController::removeMarkers(Node* node, unsigned startOffset, i
 
     // repaint the affected node
     if (docDirty && node->renderer())
-        node->renderer()->setShouldDoFullPaintInvalidation(true);
+        node->renderer()->doNotUseInvalidatePaintForWholeRendererSynchronously();
 }
 
 DocumentMarker* DocumentMarkerController::markerContainingPoint(const LayoutPoint& point, DocumentMarker::MarkerType markerType)
@@ -586,7 +586,7 @@ void DocumentMarkerController::removeMarkersFromList(MarkerMap::iterator iterato
 
     if (needsRepainting) {
         if (RenderObject* renderer = iterator->key->renderer())
-            renderer->setShouldDoFullPaintInvalidation(true);
+            renderer->doNotUseInvalidatePaintForWholeRendererSynchronously();
     }
 
     if (nodeCanBeRemoved) {
@@ -616,7 +616,7 @@ void DocumentMarkerController::repaintMarkers(DocumentMarker::MarkerTypes marker
 
             // cause the node to be redrawn
             if (RenderObject* renderer = node->renderer()) {
-                renderer->setShouldDoFullPaintInvalidation(true);
+                renderer->doNotUseInvalidatePaintForWholeRendererSynchronously();
                 break;
             }
         }
@@ -670,7 +670,7 @@ void DocumentMarkerController::shiftMarkers(Node* node, unsigned startOffset, in
 
     // repaint the affected node
     if (docDirty && node->renderer())
-        node->renderer()->setShouldDoFullPaintInvalidation(true);
+        node->renderer()->doNotUseInvalidatePaintForWholeRendererSynchronously();
 }
 
 void DocumentMarkerController::setMarkersActive(Range* range, bool active)
@@ -714,7 +714,7 @@ void DocumentMarkerController::setMarkersActive(Node* node, unsigned startOffset
 
     // repaint the affected node
     if (docDirty && node->renderer())
-        node->renderer()->setShouldDoFullPaintInvalidation(true);
+        node->renderer()->doNotUseInvalidatePaintForWholeRendererSynchronously();
 }
 
 bool DocumentMarkerController::hasMarkers(Range* range, DocumentMarker::MarkerTypes markerTypes)

Source/core/rendering/RenderObject.h

@@ -1079,6 +1079,8 @@ class RenderObject : public NoBaseWillBeGarbageCollectedFinalized<RenderObject>,
     void setNeedsOverflowRecalcAfterStyleChange();
     void markContainingBlocksForOverflowRecalc();
 
+    // FIXME: This is temporary for cases that setShouldDoFullPaintInvalidation(true) doesn't work yet.
+    void doNotUseInvalidatePaintForWholeRendererSynchronously() const { invalidatePaintForWholeRenderer(); }
     virtual LayoutRect viewRect() const;
 
     // FIXME: make this not public.