third_party/WebKit/LayoutTests/svg/clip-path/clip-path-visible-element-as-visible-shape-element.html - Issue 2375043002: Fix up that no render SVG shape when change clip-path to visible after hidden.

Unified Diff: third_party/WebKit/LayoutTests/svg/clip-path/clip-path-visible-element-as-visible-shape-element.html

Issue 2375043002: Fix up that no render SVG shape when change clip-path to visible after hidden. (Closed)

Patch Set: Issue 590153 Created 4 years, 2 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/svg/clip-path/clip-path-visible-element-as-visible-shape-element-expected.html » ('j') | third_party/WebKit/Source/core/paint/SVGShapePainter.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/LayoutTests/svg/clip-path/clip-path-visible-element-as-visible-shape-element.html

diff --git a/third_party/WebKit/LayoutTests/svg/clip-path/clip-path-visible-element-as-visible-shape-element.html b/third_party/WebKit/LayoutTests/svg/clip-path/clip-path-visible-element-as-visible-shape-element.html

new file mode 100644

index 0000000000000000000000000000000000000000..6fed12615a1a606c395fac5dd3ace3b45de01dc2

--- /dev/null

+++ b/third_party/WebKit/LayoutTests/svg/clip-path/clip-path-visible-element-as-visible-shape-element.html

@@ -0,0 +1,29 @@

+<!DOCTYPE HTML>

+<style>

+.hidden {

+ visibility: hidden;

+</style>

+<script>

+if (window.testRunner)

+ testRunner.waitUntilDone();

+window.onload = function () {

+ var classList = document.querySelector("#container").classList;

+ setTimeout(function() {

fs 2016/10/31 09:59:22 It looks like it'd be better to use the runAfterLa

hyunjunekim2 2016/10/31 13:18:30 Done.

+ classList.toggle("hidden");

+ setTimeout(function() {

+ classList.toggle("hidden");

+ if (window.testRunner)

+ testRunner.notifyDone();

+ }, 1);

+</script>

+<div id="container">

+ <svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" version="1.1" width="400" height="300">

fs 2016/10/31 09:59:22 You don't need to open/declare namespaces or speci

hyunjunekim2 2016/10/31 13:18:30 Done.

+ <clipPath id="clip">

+ <rect width="400" height="400"></rect>

+ </clipPath>

+ <rect width="100" height="100" fill="green" visibility="visible" clip-path="url(#clip)"></rect>

fs 2016/10/31 09:59:22 I think it'd make more sense to have this <rect> b

hyunjunekim2 2016/10/31 13:18:30 Done.

+ </svg>

+</div>