Xidorn Quan [:xidorn] UTC+11 Reporter
	Description • 7 years ago

The following code:
<canvas></canvas>
<script>
let canvas = document.querySelector('canvas');
let ctx = canvas.getContext('2d');
ctx.font = 'menu';
alert(ctx.font);
</script>

works as expected (shows 'menu') in Stylo, but fails (shows nothing) in Gecko.

	Xidorn Quan [:xidorn] UTC+11 Reporter
	Comment 1 • 7 years ago

dom/canvas/test/test_canvas_font_setter.html has a todo_is for this, which is unexpected pass on Stylo.

	Chris Mills [:cmills]
	Comment 2 • 7 years ago

Marking as part of documentation plan for intentional stylo differences, in case this is fixed before the stylo switchover gets into release. But not worth mentioning in the docs otherwise.

	Chris Mills [:cmills]
	Comment 3 • 7 years ago

I decided to document this anyway; I can always update the notes when this has been fixed: 

https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D#Browser_compatibility
https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/font#Quantum_CSS_notes
https://developer.mozilla.org/en-US/Firefox/Releases/57#Quantum_CSS_notes

	Xidorn Quan [:xidorn] UTC+11 Reporter
	Comment 4 • 7 years ago

I think the document is opposite on this as well. It is another bug that exists in Gecko fixed by Stylo.

	Chris Mills [:cmills]
	Comment 5 • 7 years ago

(In reply to Xidorn Quan [:xidorn] UTC+10 from comment #4)
> I think the document is opposite on this as well. It is another bug that
> exists in Gecko fixed by Stylo.

Oops. I've fixed this now. Thanks for spotting it!

	Firefox Bug Husbandry Bot
	Comment 6 • 6 years ago

https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Move_fix-optionals

	ch6ru
	Comment 7 • 6 years ago

(In reply to Firefox Product Integrity Bug Husbandry Bot (contact :emceeaich) from comment #6)
> status-firefox58: fix-optional → wontfix
> status-firefox59: --- → ?

No work in 59.0.1
http://file-store.rosalinux.ru/api/v1/file_stores/cfc9e8c9d2f43cd4a05f98934be034cf58e83568.log?show=true

	Xidorn Quan [:xidorn] UTC+11 Reporter
	Comment 8 • 6 years ago

Closing bugs existing only in the old style system.

	Xidorn Quan [:xidorn] UTC+11 Reporter
	Comment 9 • 6 years ago

(In reply to ch6ru from comment #7)
> (In reply to Firefox Product Integrity Bug Husbandry Bot (contact
> :emceeaich) from comment #6)
> > status-firefox58: fix-optional → wontfix
> > status-firefox59: --- → ?
> 
> No work in 59.0.1
> http://file-store.rosalinux.ru/api/v1/file_stores/
> cfc9e8c9d2f43cd4a05f98934be034cf58e83568.log?show=true

What is not working? The file seems to be a log and I don't know what to see from it.

	ch6ru
	Comment 10 • 6 years ago

(In reply to Xidorn Quan [:xidorn] UTC+10 from comment #9)

> What is not working? The file seems to be a log and I don't know what to see
> from it.

20:34.58 error: failed to run custom build command for `style v0.0.1 (file:///builddir/build/BUILD/firefox-59.0.1/servo/components/style)`
20:34.58 process didn't exit successfully: `/builddir/build/BUILD/obj/toolkit/library/release/build/style-5fbf2b408c8a8b6a/build-script-build` (exit code: 101)

	Xidorn Quan [:xidorn] UTC+11 Reporter
	Comment 11 • 6 years ago

(In reply to ch6ru from comment #10)
> 20:34.58 error: failed to run custom build command for `style v0.0.1
> (file:///builddir/build/BUILD/firefox-59.0.1/servo/components/style)`
> 20:34.58 process didn't exit successfully:
> `/builddir/build/BUILD/obj/toolkit/library/release/build/style-
> 5fbf2b408c8a8b6a/build-script-build` (exit code: 101)

This seems to be a build issue unrelated to this bug. Please file a new bug for this.

It seems the issue is
> 20:34.74 /builddir/build/BUILD/obj/dist/include/mozilla/Compiler.h:49:12: fatal error: 'cstddef' file not found
I'm not sure what could lead to that, probably some configuration with clang has some problem.