Do not invalidate container for outlines

progers · chromium-wpt-export-bot · commit 47f50fdf1874 · 2019-10-15T11:39:38.000-04:00
This patch reverts https://crrev.com/563452 which is causing crashes due to not marking the paint layer as needing repaint (see new testcase). The inline outline painting code was re-designed in https://crrev.com/591748 to not paint from the container, so this invalidation codepath is no longer needed. The original test for https://crrev.com/563452 is no longer crashing. Bug: 1011699 Change-Id: I42af2ffd3a5a3425fc2acaf0771a992f09adde57 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1859831 Auto-Submit: Philip Rogers <pdr@chromium.org> Commit-Queue: Aleks Totic <atotic@chromium.org> Reviewed-by: Aleks Totic <atotic@chromium.org> Cr-Commit-Position: refs/heads/master@{#705968}
diff --git a/css/css-overflow/outline-with-opacity-crash.html b/css/css-overflow/outline-with-opacity-crash.html
@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<link rel="author" title="Philip Rogers" href="mailto:pdr@chromium.org">
+<link rel="help" href="https://bugs.chromium.org/p/chromium/issues/detail?id=1011699">
+<span id="target" style="outline: 1px solid black; will-change: opacity; opacity: 0.7;">test</span>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script>
+  test(()=> {
+      target.style.background = "lightblue";
+  }, "no crash");
+</script>
