[SPv175] Disable more code when PartialRasterInvalidation is not enabled

The code disabled in this CL had some side effects, e.g. left
PartialInvalidationRect not cleared after PrePaintClean, and the next
document lifecycle might do unexpected rect mapping while no
NeedsVisualRect flag set.

Bug: 836097
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ib05ccfd439be78666771f8483a0b462128848c08
Reviewed-on: https://chromium-review.googlesource.com/1041570Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555889}

third_party/blink/renderer/core/layout/layout_object.cc

@@ -3809,7 +3809,8 @@ void LayoutObject::ClearPaintInvalidationFlags() {
 #if DCHECK_IS_ON()
   DCHECK(!ShouldCheckForPaintInvalidation() || PaintInvalidationStateIsDirty());
 #endif
-  if (!RuntimeEnabledFeatures::SlimmingPaintV175Enabled())
+  if (!RuntimeEnabledFeatures::SlimmingPaintV175Enabled() ||
+      !RuntimeEnabledFeatures::PartialRasterInvalidationEnabled())
     fragment_.SetPartialInvalidationRect(LayoutRect());
 
   ClearShouldDoFullPaintInvalidation();

third_party/blink/renderer/core/paint/object_paint_invalidator.cc

@@ -533,10 +533,12 @@ void ObjectPaintInvalidatorWithContext::InvalidateSelection(
     return;
 
   if (RuntimeEnabledFeatures::SlimmingPaintV175Enabled()) {
-    // PaintController will handle raster invalidation of the partial rect.
-    object_.GetMutableForPainting().SetPartialInvalidationRect(
-        UnionRect(object_.PartialInvalidationRect(),
-                  UnionRect(new_selection_rect, old_selection_rect)));
+    if (RuntimeEnabledFeatures::PartialRasterInvalidationEnabled()) {
+      // PaintController will handle raster invalidation of the partial rect.
+      object_.GetMutableForPainting().SetPartialInvalidationRect(
+          UnionRect(object_.PartialInvalidationRect(),
+                    UnionRect(new_selection_rect, old_selection_rect)));
+    }
   } else {
     FullyInvalidatePaint(PaintInvalidationReason::kSelection,
                          old_selection_rect, new_selection_rect);
@@ -555,13 +557,18 @@ void ObjectPaintInvalidatorWithContext::InvalidatePartialRect(
   if (rect.IsEmpty())
     return;
 
-  context_.MapLocalRectToVisualRectInBacking(object_, rect);
-  if (rect.IsEmpty())
-    return;
+  if (!RuntimeEnabledFeatures::SlimmingPaintV175Enabled() ||
+      RuntimeEnabledFeatures::PartialRasterInvalidationEnabled()) {
+    context_.MapLocalRectToVisualRectInBacking(object_, rect);
+    if (rect.IsEmpty())
+      return;
+  }
 
   if (RuntimeEnabledFeatures::SlimmingPaintV175Enabled()) {
-    // PaintController will handle raster invalidation of the partial rect.
-    object_.GetMutableForPainting().SetPartialInvalidationRect(rect);
+    if (RuntimeEnabledFeatures::PartialRasterInvalidationEnabled()) {
+      // PaintController will handle raster invalidation of the partial rect.
+      object_.GetMutableForPainting().SetPartialInvalidationRect(rect);
+    }
   } else {
     InvalidatePaintRectangleWithContext(rect,
                                         PaintInvalidationReason::kRectangle);

third_party/blink/renderer/core/paint/object_paint_invalidator_test.cc

@@ -281,6 +281,10 @@ TEST_F(ObjectPaintInvalidatorTest, InvalidatePaintRectangle) {
   EXPECT_EQ(LayoutRect(10, 10, 80, 80), target->PartialInvalidationRect());
   EXPECT_TRUE(target->MayNeedPaintInvalidation());
 
+  GetDocument().View()->UpdateLifecycleToPrePaintClean();
+  if (!RuntimeEnabledFeatures::SlimmingPaintV175Enabled() ||
+      !RuntimeEnabledFeatures::PartialRasterInvalidationEnabled())
+    EXPECT_EQ(LayoutRect(), target->PartialInvalidationRect());
   GetDocument().View()->UpdateAllLifecyclePhases();
   EXPECT_EQ(LayoutRect(), target->PartialInvalidationRect());