Skip to content

Export more code in ui_primitives to support Flutter tests.#832

Merged
polina-c merged 59 commits intoflutter:mainfrom
polina-c:uip
Mar 26, 2026
Merged

Export more code in ui_primitives to support Flutter tests.#832
polina-c merged 59 commits intoflutter:mainfrom
polina-c:uip

Conversation

@polina-c
Copy link
Copy Markdown
Collaborator

@polina-c polina-c commented Mar 26, 2026

Contributes to #825

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new ui_primitives package, which appears to be a copy of foundational Flutter code. The review comments highlight several areas that require attention before the package is published. These include updating hardcoded package names from 'flutter' to 'ui_primitives' in assertion checks and stack trace demangling, correcting bug report URLs to point to the flutter/genui repository, and addressing various TODO comments related to internal references, web stack trace parsing, UTF-8 byte length calculation, and placeholder files, as per the TEMP_README.md guidelines.

Comment thread packages/ui_primitives/lib/src/foundation/assertions.dart
Comment thread packages/ui_primitives/lib/src/dart_ui/math.dart
Comment thread packages/ui_primitives/lib/src/foundation/assertions.dart
Comment thread packages/ui_primitives/lib/src/foundation/print.dart
Comment thread packages/ui_primitives/lib/src/foundation/stack_frame.dart
Comment thread packages/ui_primitives/lib/src/foundation/stack_frame.dart
Comment thread packages/ui_primitives/lib/src/primitives/constants.dart
@polina-c polina-c changed the title Add some fixes to ui_primitives. Export more code in ui_primitives to support Flutter tests. Mar 26, 2026
@polina-c polina-c requested a review from jacobsimionato March 26, 2026 03:10
@polina-c polina-c merged commit 41c4ebc into flutter:main Mar 26, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants