Skip to content

Conversation

@HansMuller
Copy link
Contributor

@HansMuller HansMuller commented Jul 25, 2023

Force the line height of the TextStyle used in test_step.dart to be 1.0 for all Text widgets.

The test started failing after #129724 landed. Assuming that the vertical layout of the test was tight and that the change in the default font's line height is the source of the problem (see #130732 (comment)).

Fixes #130732

@HansMuller HansMuller requested a review from zanderso July 25, 2023 23:16
Copy link
Member

@zanderso zanderso left a comment

Choose a reason for hiding this comment

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

lgtm

@HansMuller HansMuller added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 26, 2023
@HansMuller
Copy link
Contributor Author

Waited for about 2 days for "Google Testing" to run/finish. Forced the bots to restart by resyncing this branch.

@auto-submit auto-submit bot merged commit c61f07c into flutter:master Jul 28, 2023
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Jul 28, 2023
flutter/flutter@dd9764e...e81907e

2023-07-28 [email protected] Update Chrome to 115, and remove `bringup: true` from skwasm benchmarks. (flutter/flutter#131430)
2023-07-28 [email protected] Fix `TimePicker` defaults for `hourMinuteTextStyle` and `dayPeriodTextColor` for Material 3 (flutter/flutter#131253)
2023-07-28 [email protected] Roll Flutter Engine from cfa5427dc452 to e97014c71014 (3 revisions) (flutter/flutter#131477)
2023-07-28 [email protected] Roll Flutter Engine from dc8618da9eb0 to cfa5427dc452 (3 revisions) (flutter/flutter#131464)
2023-07-28 [email protected] Roll Flutter Engine from b0d97ba45f46 to dc8618da9eb0 (3 revisions) (flutter/flutter#131460)
2023-07-28 [email protected] Updated integration_tests/platform_interaction test_step.dart (flutter/flutter#131301)
2023-07-28 [email protected] Roll Flutter Engine from 0713d91c2e64 to b0d97ba45f46 (1 revision) (flutter/flutter#131453)
2023-07-28 [email protected] Roll Flutter Engine from 01bb0a3ef922 to 0713d91c2e64 (2 revisions) (flutter/flutter#131451)
2023-07-28 [email protected] Roll Flutter Engine from 196474bd96b0 to 01bb0a3ef922 (3 revisions) (flutter/flutter#131447)
2023-07-27 [email protected] Roll Flutter Engine from 284771da323c to 196474bd96b0 (3 revisions) (flutter/flutter#131443)
2023-07-27 [email protected] [framework] clean up image provider documentation. (flutter/flutter#131416)
2023-07-27 [email protected] Fix template app documentation (flutter/flutter#131125)
2023-07-27 [email protected] Roll Flutter Engine from 9a935ecbf73a to 284771da323c (3 revisions) (flutter/flutter#131439)
2023-07-27 [email protected] â�¡ï¸� Add `ssh://[email protected]/flutter/flutter.git` as a standard remote (flutter/flutter#131333)
2023-07-27 [email protected] Roll Flutter Engine from f3317dbab388 to 9a935ecbf73a (1 revision) (flutter/flutter#131437)
2023-07-27 [email protected] ð��� Only format Dart files for `gen-l10n` (flutter/flutter#131232)
2023-07-27 [email protected] Roll Flutter Engine from 08050cdecf94 to f3317dbab388 (1 revision) (flutter/flutter#131432)
2023-07-27 [email protected] Preliminary PR for engine changes for Expanded/Collapsed Submenu button (flutter/flutter#131359)
2023-07-27 [email protected] Roll Flutter Engine from 9b14c382eebf to 08050cdecf94 (22 revisions) (flutter/flutter#131426)
2023-07-27 [email protected] Roll Packages from a99fc87 to f4ae933 (3 revisions) (flutter/flutter#131422)
2023-07-27 [email protected] Manual roll to engine commit 9b14c382 using Dart SDK version 3.2.x (flutter/flutter#131371)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@HansMuller HansMuller deleted the update_test_step branch July 28, 2023 17:00
LouiseHsu pushed a commit to LouiseHsu/flutter that referenced this pull request Jul 31, 2023
…r#131301)

Force the line height of the TextStyle used in test_step.dart to be 1.0 for all Text widgets.

The test started failing after flutter#129724 landed. Assuming that the vertical layout of the test was tight and that the change in the default font's line height is the source of the problem (see flutter#130732 (comment)).

Fixes flutter#130732
vashworth pushed a commit to vashworth/flutter that referenced this pull request Aug 2, 2023
…r#131301)

Force the line height of the TextStyle used in test_step.dart to be 1.0 for all Text widgets.

The test started failing after flutter#129724 landed. Assuming that the vertical layout of the test was tight and that the change in the default font's line height is the source of the problem (see flutter#130732 (comment)).

Fixes flutter#130732
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Windows_android channels_integration_test_win is flaky on Win1

2 participants