Skip to content

Make testharness.js output something in SVG documents#8137

Merged
jgraham merged 2 commits intoweb-platform-tests:masterfrom
gsnedders:testharness_svg_output
Nov 9, 2017
Merged

Make testharness.js output something in SVG documents#8137
jgraham merged 2 commits intoweb-platform-tests:masterfrom
gsnedders:testharness_svg_output

Conversation

@gsnedders
Copy link
Member

@gsnedders gsnedders commented Nov 9, 2017

This change is Reviewable

@gsnedders gsnedders requested a review from jgraham November 9, 2017 22:00
@wpt-pr-bot wpt-pr-bot requested a review from ayg November 9, 2017 22:00
} else {
var is_svg = false;
try {
is_svg = output_document.documentElement instanceof SVGSVGElement;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't work if output_document != document, right?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jgraham oh that's true

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jgraham fixed

@ghost
Copy link

ghost commented Nov 9, 2017

Build PASSED

Started: 2017-11-09 22:35:13
Finished: 2017-11-09 22:39:44

View more information about this build on:

@jgraham jgraham merged commit 0ca66ce into web-platform-tests:master Nov 9, 2017
@gsnedders gsnedders deleted the testharness_svg_output branch November 9, 2017 23:09
@gsnedders gsnedders restored the testharness_svg_output branch May 28, 2018 16:03
@gsnedders gsnedders deleted the testharness_svg_output branch March 17, 2023 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants