New issue
Advanced search Search tips

Issue 815279 link

Starred by 3 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 2
Type: Feature



Sign in to add a comment

Canvas rendering context does not respect dynamic changes to the user font set

Reported by chromium...@gmail.com, Feb 23 2018

Issue description

VERSION
Chrome Version: 66.0.3353.0 (Official Build) canary (64-bit)
Operating System: Mac

REPRODUCTION CASE
for more details https://bugzilla.mozilla.org/show_bug.cgi?id=950590
 
Canvas.html
1.6 KB View Download
Screen Shot 2018-02-23 at 20.43.49.png
87.3 KB View Download

Comment 1 by wfh@chromium.org, Feb 23 2018

Components: Blink>Fonts
I think this was filed as security in mozilla because of https://bugzilla.mozilla.org/show_bug.cgi?id=950000 - but I'm not sure our font code has the same codepath. Can you explain how exactly this issue would pose a security risk to Chrome users? Thanks!
Actually I'm not really sure about this bug and I'm not familiar with fonts, I filed this bug because in comment 1 said: "Chrome also appears to get this wrong" and I tested the test case on Chrome and I didn't get the same result on Firefox. Please you can close this bug if is it an invalid bug :-)

Comment 3 by wfh@chromium.org, Feb 23 2018

Cc: layout-dev@chromium.org
Labels: -Type-Bug-Security -Restrict-View-SecurityTeam Pri-1 Type-Bug
Summary: Canvas rendering context does not respect dynamic changes to the user font set (was: Security: Canvas rendering context does not respect dynamic changes to the user font set)
no this seems like a valid bug (thanks for reporting) but I don't think it has the same security implications. I'll transmogrify this into a feature bug and see if blink team can look at it.
Components: Blink>Canvas
Labels: Needs-Triage-M66
Labels: -Pri-1 Triaged-ET M-66 FoundIn-66 Target-66 OS-Linux OS-Mac OS-Windows Pri-2
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on Mac 10.13.3, Win-10 and Ubuntu 14.04 using chrome reported version #66.0.3353.0.
This is a non-regression issue as it is observed from M60 old builds. 

Hence, marking it as untriaged to get more inputs from dev team.

Thanks...!!

Comment 7 by junov@chromium.org, Feb 26 2018

Labels: -Type-Bug -M-66 -Target-66 -Needs-Triage-M66 OS-Android OS-Chrome Type-Feature
Owner: fs...@chromium.org
Status: Assigned (was: Untriaged)
This would be very simple to fix, but I believe the current behavior is spec compliant:

https://html.spec.whatwg.org/multipage/canvas.html#dom-context-2d-font

Basically: fonts get resolved when the 'font' attribute is set.

If we think there is a fundamental problem with this way of doing things, then we should first discuss amending the spec.

Comment 8 Deleted

Cc: fs...@chromium.org
Owner: ----
Status: Available (was: Assigned)

Sign in to add a comment