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

Conversation

@reidbaker
Copy link
Contributor

@reidbaker reidbaker commented May 26, 2023

#flutter/flutter/127682
The plurality of android users are on api 33. We should be testing on our most used android api.

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides].
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See [testing the engine] for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the [CLA].
  • All existing and new tests are passing.

@johnmccutchan
Copy link
Contributor

Could you update the description with more context? e.g. the motivation of the change and maybe elaborate on what you mean by "see what happens".

@johnmccutchan
Copy link
Contributor

and also LGTM

@reidbaker reidbaker changed the title try to set device to version 33 to see what happens Update engine testing to use api 33 for android tests. May 26, 2023
@reidbaker reidbaker added the autosubmit Merge PR when tree becomes green via auto submit App label May 30, 2023
@auto-submit auto-submit bot merged commit a8d6252 into flutter:main May 30, 2023
@reidbaker reidbaker deleted the i127682-android-api-33-test-support branch May 30, 2023 20:50
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 30, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 30, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 30, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request May 30, 2023
…127890)

flutter/engine@a6d3124...1ba8091

2023-05-30 [email protected] Handle a11y focus event on Ios and android (flutter/engine#41777)
2023-05-30 [email protected] Fix unitialized SkRect (flutter/engine#42403)
2023-05-30 [email protected] Update engine testing to use api 33 for android tests.  (flutter/engine#42351)
2023-05-30 [email protected] Reland: "Adding app lifecycle notification for macOS and Linux, add `hidden` state." (#40542) (flutter/engine#42418)
2023-05-30 [email protected] [Impeller] Fixed GlyphAtlasWithLotsOfdUniqueGlyphSize (flutter/engine#42423)
2023-05-30 [email protected] Remove legacy xcode properties/dependencies (flutter/engine#42368)
2023-05-30 [email protected] Roll Skia from bab82f14da1a to fed417995ba9 (7 revisions) (flutter/engine#42424)
2023-05-30 [email protected] [Impeller] removed vulkan unit test csv (flutter/engine#42354)
2023-05-30 [email protected] Roll Skia from d22106cc40a0 to bab82f14da1a (7 revisions) (flutter/engine#42420)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
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 Flutter: 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
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants