Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Refactor - Create Renderer interface for CanvasKit/HTML switching#35502

Merged
auto-submit[bot] merged 18 commits into
flutter-team-archive:mainfrom
eyebrowsoffire:web_renderer
Aug 23, 2022
Merged

Refactor - Create Renderer interface for CanvasKit/HTML switching#35502
auto-submit[bot] merged 18 commits into
flutter-team-archive:mainfrom
eyebrowsoffire:web_renderer

Conversation

@eyebrowsoffire

Copy link
Copy Markdown
Contributor

This is a refactor which creates a Renderer interface that is implemented for both CanvasKit and HTML. This will allow us to more easily add and integrate a third renderer.

@flutter-dashboard flutter-dashboard Bot added the platform-web Code specifically for the web engine label Aug 18, 2022
@eyebrowsoffire
eyebrowsoffire requested a review from yjbanov August 18, 2022 17:50
Comment thread lib/web_ui/lib/src/engine/renderer.dart Outdated
Comment thread lib/web_ui/lib/src/engine/canvaskit/renderer.dart Outdated
Comment thread lib/web_ui/lib/src/engine/canvaskit/renderer.dart Outdated
Comment thread lib/web_ui/lib/src/engine/canvaskit/embedded_views.dart Outdated
Comment thread lib/web_ui/lib/src/engine/canvaskit/font_fallbacks.dart Outdated
Comment thread lib/web_ui/lib/src/engine/canvaskit/fonts.dart Outdated
Comment thread lib/web_ui/lib/src/engine/renderer.dart

@mdebbar mdebbar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! All my comments are nits and minor suggestions.

Comment thread lib/web_ui/lib/painting.dart Outdated
Comment thread lib/web_ui/lib/painting.dart Outdated
Comment thread lib/web_ui/lib/src/engine/canvaskit/font_fallbacks.dart Outdated
Comment thread lib/web_ui/lib/src/engine/canvaskit/skia_object_cache.dart Outdated
Comment thread lib/web_ui/lib/src/engine/canvaskit/text.dart Outdated
Comment thread lib/web_ui/test/canvaskit/common.dart Outdated
Comment thread web_sdk/web_engine_tester/lib/golden_tester.dart
Comment thread lib/web_ui/lib/src/engine/canvaskit/renderer.dart Outdated
Comment thread lib/web_ui/lib/src/engine/html/renderer.dart Outdated
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 24, 2022
zanderso pushed a commit to flutter/flutter that referenced this pull request Aug 24, 2022
* 68beed7 Refactor - Create `Renderer` interface for CanvasKit/HTML switching (flutter-team-archive/engine#35502)

* 2ecccf1 [Impeller] Enforce a minimum stroke width (flutter-team-archive/engine#35576)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App platform-web Code specifically for the web engine

Development

Successfully merging this pull request may close these issues.

3 participants