Comments (11)
First of all, thank you very much for your work. Also, have you tried submitting these patches you wrote to Google for review?
from thorium.
The last comments in both threads are mine :
https://bugs.chromium.org/p/chromium/issues/detail?id=306078
https://issues.chromium.org/40336440 . In addition, I am in some other mailing lists where the relevant google engineers frequent too. Consider the work is mostly done by them and the patches do very little beyond flipping some options, I think their choice is more a strategy against Firefox and Webkit than technical. (As in make it work only as a competitive target to try to do a better(?) alternative).
from thorium.
Have you verified that the relevant support actually takes effect after applying these patches? In addition, I took a look at the patches you wrote, and there seem to be some inconsistencies in them that do not comply with Google's coding standards.
from thorium.
Testing and verification are done via https://github.com/HinTak/Qt6WE-OT-SVG . As for coding standard, please feel free to comment further. Yes, it is against m118, slightly older.
from thorium.
You only need to use the git cl format
command after editing the Chromium source code to automatically reformat the code and comply with Google's coding standards. The prerequisite for running the above command is that you need to deploy Chromium's depot_tools.
from thorium.
Okay, thanks for the tips. I only work with skia in git; chromium against snapshot tar balls. I 'll think about how to do the formatting on github ci.
from thorium.
I probably should put these two links next to the patches (they are on the minimal web browser script README) - just visit them before and after applying the patches to see the difference:
https://pixelambacht.nl/chromacheck/
https://yoksel.github.io/color-fonts-demo/
(Or with a different browser - Firefox/ webkitgtk both handle OT-SVG quite well, but missing COLRv1 and others which chrome supports)
from thorium.
If you have a git repo handy, can you git cl format
then post the result? If it is just white space, I 'll up date the patches.
from thorium.
We don't need to reformat the code yet, I just gave you a reminder. I saw in your patches that the files that need to be modified involve components such as ots, skia, and blink. Have you discussed it with the developers of ots, skia, and blink before or submitted patches to them?
By the way, due to some potential impacts (security, binary size, W3C specifications, etc.), we should modify the relevant files with caution.
from thorium.
The relevant people are all CC'ed on those two issue URLs, as well as subscribers of a few mailing lists on which I posted the work. This has been going on for 8 months. If they want to say something, they'd have done so by now.
from thorium.
The skia part of the patch has gotten into skia-python 6 months ago (and I have been invited as co-maintainer/releases since).
from thorium.
Related Issues (20)
- Revert Chrome Refresh 2023 or keep it avoidable HOT 1
- Detected: Trojan:Win32/Znyonm HOT 10
- Thorium overwrites any Web Data file it did not generate HOT 2
- macos: Restore tabs to appropriate Mission Control desktop Space HOT 2
- What should I do to install Thorium on my Windows 11? HOT 4
- `chrome://flags/#disable-thorium-icons` - not all icons are grey HOT 1
- updating breaks browser HOT 3
- Distribution via flatpaks/better support for distros outside of Debian-based ones HOT 2
- Some mistakes about the TopBar icon. HOT 2
- AppArmor Error HOT 3
- Detected: Trojan:Win32/Acll in chrome_proxy.exe HOT 1
- Can not disable Global Media Control any longer HOT 5
- update to 124.0.6367.119 HOT 3
- SSE3 Build for X86 32bit systems? HOT 10
- New patch of Cromite browser to enbale by default energy battery saver HOT 4
- geforce now "streaming games is not supported in this browser" HOT 3
- Cant save a facebook webpage offline as html, the three "save as" functions dont work. HOT 2
- scaling problem HOT 1
- Browser de-saturates and alters colors?
- I have a SERIOUS Issue HOT 9
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from thorium.