third_party/WebKit/LayoutTests/svg/dom/title-in-shadow-tree-without-any-title-in-use.html - Issue 2706583002: Fix the way <title> is read under <use> shadow tree

Unified Diff: third_party/WebKit/LayoutTests/svg/dom/title-in-shadow-tree-without-any-title-in-use.html

Issue 2706583002: Fix the way <title> is read under <use> shadow tree (Closed)

Patch Set: Add layout test Created 3 years, 10 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/dom/title-in-shadow-tree-without-any-title-in-use-expected.txt » ('j') | third_party/WebKit/Source/core/svg/SVGUseElement.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/LayoutTests/svg/dom/title-in-shadow-tree-without-any-title-in-use.html

diff --git a/third_party/WebKit/LayoutTests/svg/dom/title-in-shadow-tree-without-any-title-in-use.html b/third_party/WebKit/LayoutTests/svg/dom/title-in-shadow-tree-without-any-title-in-use.html

new file mode 100644

index 0000000000000000000000000000000000000000..12f33355a6b65d330612d635835b2ca969ccc1fd

--- /dev/null

+++ b/third_party/WebKit/LayoutTests/svg/dom/title-in-shadow-tree-without-any-title-in-use.html

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

+<!DOCTYPE HTML>

+<template id="template">

+ <svg width="100" height="100">

+ <symbol id="greenSquare">

+ <title>PASS: This text should appear as a tooltip.</title>

+ <rect width="50" height="50" fill="green">

+ <title>FAIL: This text should not appear as a tooltip.</title>

+ </rect>

+ </symbol>

+ <use x="50" y="50" xlink:href="#greenSquare">

+ </use>

fs 2017/03/01 09:29:05 Nit: Can move this to the previous line.

+ </svg>

+</template>

+<script>

+ document.body.createShadowRoot().appendChild(template.content.cloneNode(true));

fs 2017/03/01 09:29:05 Should test this as a regular <use> (outside of an

+ onload = function() {

+ if (window.eventSender) {

+ eventSender.dragMode = false;

+ eventSender.mouseMoveTo(75, 75);

+ document.write(testRunner.tooltipText);

+ if (window.testRunner)

+ testRunner.dumpAsText();

fs 2017/03/01 09:29:05 It should be possible (and preferable) to write th

mrunal 2017/03/02 03:37:43 Done. I have removed this test and added testharne

fs 2017/03/02 08:45:07 ManualTests are manually, so I guess in general th

+ }

+</script>

+FAIL: Test did not run.