docs(readme): add comparisons#34
Merged
Merged
Conversation
jmagly
added a commit
to jmagly/carbonyl
that referenced
this pull request
Apr 13, 2026
Phase 1 of M135→M147 catch-up (fathyb#33, fathyb#34). Rebases all 24 carbonyl patches from M135 (135.0.7049.84) to M140 (140.0.7339.264). Patch conflicts resolved (7): - 0002: dropped CONTENT_ENABLE_LEGACY_IPC blocks (removed in M140) - 0006: kept carbonyl DPI scaling (M140 simplified SetBoundsInPixels) - 0007: kept disabled underline/overline (M140 added TextDecorationSkipInk) - 0008: kept monospace font forcing (M140 refactored sibling functions) - 0009: kept both HeadlessWindowDelegate + WebContentsObserver bases - 0013: took M140 side for printing_context_mac.mm USE_CUPS removal Build fixes for M140 API drift: - Skia drawPoints: size_t + SkPoint[] → SkSpan<const SkPoint> - Skia getRelativeTransform: returns SkM44 (use .asM33()) - HeadlessWebContentsImpl: added using decl for OnVisibilityChanged - font.h: re-added Font::DrawText(TextRun) overload declarations - font.cc: added #include "cc/paint/paint_canvas.h" Verified: ninja 3676/3676, audit Cat A inert, smoke test 3/3. Refs fathyb#34
jmagly
added a commit
to jmagly/carbonyl
that referenced
this pull request
Apr 13, 2026
- Add M135→M140 upgrade section to changelog with conflict resolution and build fix details - Update readme: M135 → M140, patch phases 4 → 5 - Update MAINTENANCE: version string, base commits, patch notes, GN args (enable_nacl removed in M140) Refs fathyb#34
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #31