-
Notifications
You must be signed in to change notification settings - Fork 10
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
Check fonts leak #52
Comments
it seems that only windows is interested. |
for android check https://bugs.chromium.org/p/chromium/issues/detail?id=1115064 https://chromium-review.googlesource.com/c/chromium/src/+/2347916 https://chromium-review.googlesource.com/c/chromium/src/+/2375329 |
|
it seems that brave has changed the logic regarding the fallback list of fonts: font_fallback_list.cc and third_party-blink-renderer-platform-fonts-font_fallback_list.cc.patch |
ok, it seems that the problem is fonts related to the user's language. |
https://bugs.chromium.org/p/chromium/issues/detail?id=366983#c9 so it is not possible for me to act directly in skia, since some non-standard fonts such as 'Arial Narrow' are considered by DirectWrite (active by default) as font-family 'Arial' variant_stretch = 'Narrow' (Condensed = 7). a possible alternative is to filter the list of fonts when loading the (custom) collection in DWriteFontCollectionProxy::CreateEnumeratorFromKey, but I would have to wire the ttf file name and that might not be the best solution. |
nooo, for the same reason blink doesn't work either!
meanwhile, i guess only that is the correct solution. |
definitively found the solution, which is to retrieve the gdi name of the font via directwrite. |
For now, I have decided to only insert the flag |
@GY8VSdYYzvL8-K6T
what that patch does is to inhibit the use of fonts that do not appear in the list of standard fonts in the respective platforms (by default user language), to mitigate the possibility of fonts being detected in the device, installed due to user activity (e.g. in windows, some apps also install certain fonts, see office for example). |
for those who want to check, these are the data from my browser (viewport size protection off), which should be the same for everyone in my area (eu) regardless of the fonts installed on a windows machine source https://browserleaks.com/fonts the first column should be a value indicating the size of the rectangle containing a given constant string
|
see https://browserleaks.com/fonts
https://www.deviceinfo.me/
https://abrahamjuliot.github.io/creepjs/tests/fonts.html
The text was updated successfully, but these errors were encountered: