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

Cleanup the shell test, removing unused code #54238

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

matanlurey
Copy link
Contributor

Guess we'll find out!

Copy link
Member

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

Should be used, see shell_unittests.cc for the configuration.SetEntrypoint calls

@matanlurey
Copy link
Contributor Author

Should be used, see shell_unittests.cc for the configuration.SetEntrypoint calls

Looks like the code I deleted doesn't run on CI, at best?

@matanlurey matanlurey requested a review from jonahwilliams July 31, 2024 16:55
@matanlurey matanlurey marked this pull request as ready for review July 31, 2024 16:55
@matanlurey matanlurey changed the title Was this code dead or undead? Cleanup the shell test, removing unused code Jul 31, 2024
Copy link
Member

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@matanlurey matanlurey added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 31, 2024
@auto-submit auto-submit bot merged commit 4f9dbf5 into flutter:main Jul 31, 2024
27 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jul 31, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jul 31, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jul 31, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jul 31, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Aug 1, 2024
…sions) (#152648)

Manual roll requested by [email protected]

flutter/engine@1633272...32f7888

2024-07-31 [email protected] Roll Skia from 431b57332241 to b5ad5bf3696d (3 revisions) (flutter/engine#54259)
2024-07-31 [email protected] Roll Fuchsia Linux SDK from cYjTzxe0MskG7PtkB... to uF76DfQgigt4utdBv... (flutter/engine#54258)
2024-07-31 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Impeller] Reland: Implement draw order optimization. (#54215)" (flutter/engine#54261)
2024-07-31 [email protected] Roll Skia from cd58e87c06f9 to 431b57332241 (2 revisions) (flutter/engine#54257)
2024-07-31 [email protected] [iOS][macOS] Eliminate use of bitcode_strip (flutter/engine#54254)
2024-07-31 [email protected] Roll Skia from e9e423457655 to cd58e87c06f9 (5 revisions) (flutter/engine#54253)
2024-07-31 [email protected] Cleanup the shell test, removing unused code (flutter/engine#54238)
2024-07-31 [email protected] [web] rename dialog to route to match the framework (flutter/engine#54228)
2024-07-31 [email protected] Roll Dart SDK from edace067d950 to 5df6a6e0c037 (1 revision) (flutter/engine#54252)
2024-07-31 [email protected] [Impeller] Reland: Implement draw order optimization. (flutter/engine#54215)
2024-07-31 [email protected] Migrate the remaining real pub packages to pub workspaces. (flutter/engine#54232)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from cYjTzxe0MskG to uF76DfQgigt4

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://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
TytaniumDev pushed a commit to TytaniumDev/flutter that referenced this pull request Aug 7, 2024
…sions) (flutter#152648)

Manual roll requested by [email protected]

flutter/engine@1633272...32f7888

2024-07-31 [email protected] Roll Skia from 431b57332241 to b5ad5bf3696d (3 revisions) (flutter/engine#54259)
2024-07-31 [email protected] Roll Fuchsia Linux SDK from cYjTzxe0MskG7PtkB... to uF76DfQgigt4utdBv... (flutter/engine#54258)
2024-07-31 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Impeller] Reland: Implement draw order optimization. (flutter#54215)" (flutter/engine#54261)
2024-07-31 [email protected] Roll Skia from cd58e87c06f9 to 431b57332241 (2 revisions) (flutter/engine#54257)
2024-07-31 [email protected] [iOS][macOS] Eliminate use of bitcode_strip (flutter/engine#54254)
2024-07-31 [email protected] Roll Skia from e9e423457655 to cd58e87c06f9 (5 revisions) (flutter/engine#54253)
2024-07-31 [email protected] Cleanup the shell test, removing unused code (flutter/engine#54238)
2024-07-31 [email protected] [web] rename dialog to route to match the framework (flutter/engine#54228)
2024-07-31 [email protected] Roll Dart SDK from edace067d950 to 5df6a6e0c037 (1 revision) (flutter/engine#54252)
2024-07-31 [email protected] [Impeller] Reland: Implement draw order optimization. (flutter/engine#54215)
2024-07-31 [email protected] Migrate the remaining real pub packages to pub workspaces. (flutter/engine#54232)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from cYjTzxe0MskG to uF76DfQgigt4

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://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Buchimi pushed a commit to Buchimi/flutter that referenced this pull request Sep 2, 2024
…sions) (flutter#152648)

Manual roll requested by [email protected]

flutter/engine@1633272...32f7888

2024-07-31 [email protected] Roll Skia from 431b57332241 to b5ad5bf3696d (3 revisions) (flutter/engine#54259)
2024-07-31 [email protected] Roll Fuchsia Linux SDK from cYjTzxe0MskG7PtkB... to uF76DfQgigt4utdBv... (flutter/engine#54258)
2024-07-31 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Impeller] Reland: Implement draw order optimization. (flutter#54215)" (flutter/engine#54261)
2024-07-31 [email protected] Roll Skia from cd58e87c06f9 to 431b57332241 (2 revisions) (flutter/engine#54257)
2024-07-31 [email protected] [iOS][macOS] Eliminate use of bitcode_strip (flutter/engine#54254)
2024-07-31 [email protected] Roll Skia from e9e423457655 to cd58e87c06f9 (5 revisions) (flutter/engine#54253)
2024-07-31 [email protected] Cleanup the shell test, removing unused code (flutter/engine#54238)
2024-07-31 [email protected] [web] rename dialog to route to match the framework (flutter/engine#54228)
2024-07-31 [email protected] Roll Dart SDK from edace067d950 to 5df6a6e0c037 (1 revision) (flutter/engine#54252)
2024-07-31 [email protected] [Impeller] Reland: Implement draw order optimization. (flutter/engine#54215)
2024-07-31 [email protected] Migrate the remaining real pub packages to pub workspaces. (flutter/engine#54232)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from cYjTzxe0MskG to uF76DfQgigt4

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://issues.skia.org/issues/new?component=1389291&template=1850622

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.

2 participants