Skip to content

Remove dupe media/CanvasTest.ttf and unused fonts/CanvasTest.sfd #5678

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 25, 2017

Conversation

foolip
Copy link
Member

@foolip foolip commented Apr 24, 2017

Most tests that use either already use fonts/CanvasTest.ttf
No tests at all use fonts/CanvasTest.sdf


This change is Reviewable

@wpt-pr-bot
Copy link
Collaborator

Notifying @gsnedders and @hillbrad. (Learn how reviewing works.)

@ghost
Copy link

ghost commented Apr 24, 2017

View the complete job log.

Lint

Passed

@ghost
Copy link

ghost commented Apr 24, 2017

View the complete job log.

Firefox (nightly channel)

Testing web-platform-tests at revision 2c3f4e7
Using browser at version BuildID 20170423100233; SourceStamp 070fc3c2f8400c09f1037f5444c3d33b499e7128
Starting 10 test iterations
All results were stable

All results

4 tests ran
/preload/download-resources.html
Subtest Results Messages
OK
Makes sure that preloaded resources are downloaded FAIL assert_true: Preload not supported expected true got false
/preload/onload-event.html
Subtest Results Messages
OK
Makes sure that preloaded resources trigger the onload event FAIL assert_true: Preload not supported expected true got false
/preload/preload-with-type.html
Subtest Results Messages
OK
Makes sure that preloaded resources with a type attribute trigger the onload event FAIL assert_true: Preload not supported expected true got false
/preload/single-download-preload.html
Subtest Results Messages
OK
Makes sure that preloaded resources are not downloaded again when used FAIL assert_true: Preload not supported expected true got false

@ghost
Copy link

ghost commented Apr 24, 2017

View the complete job log.

Chrome (unstable channel)

Testing web-platform-tests at revision 2c3f4e7
Using browser at version 59.0.3071.15 dev
Starting 10 test iterations
All results were stable

All results

4 tests ran
/preload/download-resources.html
Subtest Results Messages
OK
Makes sure that preloaded resources are downloaded PASS
/preload/onload-event.html
Subtest Results Messages
OK
Makes sure that preloaded resources trigger the onload event PASS
/preload/preload-with-type.html
Subtest Results Messages
OK
Makes sure that preloaded resources with a type attribute trigger the onload event PASS
/preload/single-download-preload.html
Subtest Results Messages
OK
Makes sure that preloaded resources are not downloaded again when used PASS

Most tests that use either already use fonts/CanvasTest.ttf
No tests at all use fonts/CanvasTest.sdf
@foolip foolip force-pushed the rm-media-CanvasTest branch from ce295b1 to 25c51d8 Compare April 25, 2017 05:20
@foolip foolip changed the title Remove dupe media/CanvasTest.ttd and unused fonts/CanvasTest.sfd Remove dupe media/CanvasTest.ttf and unused fonts/CanvasTest.sfd Apr 25, 2017
@gsnedders gsnedders merged commit e5474e0 into master Apr 25, 2017
@gsnedders gsnedders deleted the rm-media-CanvasTest branch April 25, 2017 12:37
MXEBot pushed a commit to mirror/chromium that referenced this pull request Apr 26, 2017
The copy in media/ is identical, and has been removed upstream:
web-platform-tests/wpt#5678

[email protected],[email protected]

Review-Url: https://codereview.chromium.org/2838673002
Cr-Commit-Position: refs/heads/master@{#466961}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants