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

Started caching HandleGLES's hash and made them immutable #56800

Merged
auto-submit merged 2 commits into from
Nov 26, 2024
Merged

Started caching HandleGLES's hash and made them immutable #56800

auto-submit merged 2 commits into from
Nov 26, 2024

Conversation

@gaaclarke
Copy link
Member

@gaaclarke gaaclarke commented Nov 25, 2024

flutter/flutter#159177
test exempt: no functional change, performance

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 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.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

jonahwilliams
jonahwilliams approved these changes Nov 25, 2024
View reviewed changes
Copy link
Contributor

@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

@gaaclarke
Copy link
Member Author

gaaclarke commented Nov 25, 2024

rebase to resolve conflicts

@gaaclarke gaaclarke added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 25, 2024
@auto-submit
Copy link
Contributor

auto-submit bot commented Nov 26, 2024

auto label is removed for flutter/engine/56800, due to - The status or check suite Mac mac_clang_tidy has failed. Please fix the issues identified (or deflake) before re-applying this label.

  • The status or check suite Linux linux_fuchsia_tests has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Nov 26, 2024
@gaaclarke gaaclarke added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 26, 2024
@auto-submit auto-submit bot merged commit 8ad5709 into flutter:main Nov 26, 2024
29 checks passed
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Nov 26, 2024
Merged
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 26, 2024
@engine-flutter-autoroll
8ad5709 Started caching HandleGLES's hash and made them immutable (fl…
78aa825 
…utter/engine#56800)
github-merge-queue bot pushed a commit to flutter/flutter that referenced this pull request Nov 26, 2024
@engine-flutter-autoroll
Roll Flutter Engine from fe45a66 to fb64399 (12 revisions) (#159461)
1e824af 
flutter/engine@fe45a66...fb64399

2024-11-26 chris@bracken.jp iOS: Migrate PlatformViewsController to
Objective-C (flutter/engine#56790)
2024-11-26 30870216+gaaclarke@users.noreply.github.com Started caching
HandleGLES's hash and made them immutable (flutter/engine#56800)
2024-11-26 98614782+auto-submit[bot]@users.noreply.github.com Reverts
"[iOS] Full keyboard access scrolling (#56606)" (flutter/engine#56802)
2024-11-26 skia-flutter-autoroll@skia.org Roll Skia from 12c8bd6ac1d9 to
c1c8ff84997c (14 revisions) (flutter/engine#56801)
2024-11-25 jonahwilliams@google.com [Impeller] better handle allocation
herustics of Android slide in page transition. (flutter/engine#56762)
2024-11-25 chris@bracken.jp iOS: Eliminate logging of non-zero origin
platformviews (flutter/engine#56796)
2024-11-25 30870216+gaaclarke@users.noreply.github.com [impeller] gles:
started storing the number of handle deletions to avoid reallocation
(flutter/engine#56799)
2024-11-25 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from
9o0fWa2xVhmxV6Mtn... to 50xtjbMWWrqay_7m_... (flutter/engine#56795)
2024-11-25 skia-flutter-autoroll@skia.org Roll Dart SDK from
df716eaa6ed2 to 4b49546a1dfa (1 revision) (flutter/engine#56793)
2024-11-25 31859944+LongCatIsLooong@users.noreply.github.com [iOS] Full
keyboard access scrolling (flutter/engine#56606)
2024-11-25 jonahwilliams@google.com [android] remove fml_check from
surface_texture_external_texture (flutter/engine#56760)
2024-11-25 30870216+gaaclarke@users.noreply.github.com removed unused
variable for skia initialization (flutter/engine#56791)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from 9o0fWa2xVhmx to 50xtjbMWWrqa

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 aaclarke@google.com,zra@google.com 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
@auto-submit auto-submit bot mentioned this pull request Nov 26, 2024
Merged
github-merge-queue bot pushed a commit to flutter/flutter that referenced this pull request Nov 26, 2024
@auto-submit
Reverts "Roll Flutter Engine from fe45a66 to fb64399 (12 revisions) (#…
e9fe719 
…159461)" (#159498)

<!-- start_original_pr_link -->
Reverts: #159461
<!-- end_original_pr_link -->
<!-- start_initiating_author -->
Initiated by: yjbanov
<!-- end_initiating_author -->
<!-- start_revert_reason -->
Reason for reverting: engine regression
#159497
<!-- end_revert_reason -->
<!-- start_original_pr_author -->
Original PR Author: engine-flutter-autoroll
<!-- end_original_pr_author -->

<!-- start_reviewers -->
Reviewed By: {fluttergithubbot}
<!-- end_reviewers -->

<!-- start_revert_body -->
This change reverts the following previous change:

flutter/engine@fe45a66...fb64399

2024-11-26 chris@bracken.jp iOS: Migrate PlatformViewsController to
Objective-C (flutter/engine#56790)
2024-11-26 30870216+gaaclarke@users.noreply.github.com Started caching
HandleGLES's hash and made them immutable (flutter/engine#56800)
2024-11-26 98614782+auto-submit[bot]@users.noreply.github.com Reverts
"[iOS] Full keyboard access scrolling (#56606)" (flutter/engine#56802)
2024-11-26 skia-flutter-autoroll@skia.org Roll Skia from 12c8bd6ac1d9 to
c1c8ff84997c (14 revisions) (flutter/engine#56801)
2024-11-25 jonahwilliams@google.com [Impeller] better handle allocation
herustics of Android slide in page transition. (flutter/engine#56762)
2024-11-25 chris@bracken.jp iOS: Eliminate logging of non-zero origin
platformviews (flutter/engine#56796)
2024-11-25 30870216+gaaclarke@users.noreply.github.com [impeller] gles:
started storing the number of handle deletions to avoid reallocation
(flutter/engine#56799)
2024-11-25 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from
9o0fWa2xVhmxV6Mtn... to 50xtjbMWWrqay_7m_... (flutter/engine#56795)
2024-11-25 skia-flutter-autoroll@skia.org Roll Dart SDK from
df716eaa6ed2 to 4b49546a1dfa (1 revision) (flutter/engine#56793)
2024-11-25 31859944+LongCatIsLooong@users.noreply.github.com [iOS] Full
keyboard access scrolling (flutter/engine#56606)
2024-11-25 jonahwilliams@google.com [android] remove fml_check from
surface_texture_external_texture (flutter/engine#56760)
2024-11-25 30870216+gaaclarke@users.noreply.github.com removed unused
variable for skia initialization (flutter/engine#56791)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from 9o0fWa2xVhmx to 50xtjbMWWrqa

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 aaclarke@google.com,zra@google.com 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

<!-- end_revert_body -->

Co-authored-by: auto-submit[bot] <flutter-engprod-team@google.com>
@engine-flutter-autoroll engine-flutter-autoroll mentioned this pull request Nov 26, 2024
Merged
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 26, 2024
@engine-flutter-autoroll
8ad5709 Started caching HandleGLES's hash and made them immutable (fl…
11c9abb 
…utter/engine#56800)
github-merge-queue bot pushed a commit to flutter/flutter that referenced this pull request Nov 26, 2024
@engine-flutter-autoroll
Roll Flutter Engine from fe45a66 to bd165dc (23 revisions) (#159504)
830ec5a 
flutter/engine@fe45a66...bd165dc

2024-11-26 chris@bracken.jp Revert "iOS: Migrate PlatformViewsController
to Objective-C (#56790)" (flutter/engine#56817)
2024-11-26 chris@bracken.jp iOS: Rename FlutterPlatformViews_Internal.mm
(flutter/engine#56816)
2024-11-26 chris@bracken.jp iOS: Eliminate global in platformviews
controller (flutter/engine#56805)
2024-11-26 skia-flutter-autoroll@skia.org Roll Skia from a276978ba7c8 to
f149f852c70a (1 revision) (flutter/engine#56812)
2024-11-26 skia-flutter-autoroll@skia.org Roll Skia from d7a267d88fd6 to
a276978ba7c8 (1 revision) (flutter/engine#56811)
2024-11-26 skia-flutter-autoroll@skia.org Roll Skia from 8d9d892657a7 to
d7a267d88fd6 (1 revision) (flutter/engine#56810)
2024-11-26 skia-flutter-autoroll@skia.org Roll Dart SDK from
bdb76c714009 to ca02d403f1a8 (1 revision) (flutter/engine#56809)
2024-11-26 skia-flutter-autoroll@skia.org Roll Skia from b697dd1b03b2 to
8d9d892657a7 (1 revision) (flutter/engine#56808)
2024-11-26 skia-flutter-autoroll@skia.org Roll Skia from c1c8ff84997c to
b697dd1b03b2 (1 revision) (flutter/engine#56807)
2024-11-26 chris@bracken.jp iOS: Delete
FlutterPlatformViewsController.layerPoolSize (flutter/engine#56806)
2024-11-26 skia-flutter-autoroll@skia.org Roll Dart SDK from
4b49546a1dfa to bdb76c714009 (1 revision) (flutter/engine#56803)
2024-11-26 chris@bracken.jp iOS: Migrate PlatformViewsController to
Objective-C (flutter/engine#56790)
2024-11-26 30870216+gaaclarke@users.noreply.github.com Started caching
HandleGLES's hash and made them immutable (flutter/engine#56800)
2024-11-26 98614782+auto-submit[bot]@users.noreply.github.com Reverts
"[iOS] Full keyboard access scrolling (#56606)" (flutter/engine#56802)
2024-11-26 skia-flutter-autoroll@skia.org Roll Skia from 12c8bd6ac1d9 to
c1c8ff84997c (14 revisions) (flutter/engine#56801)
2024-11-25 jonahwilliams@google.com [Impeller] better handle allocation
herustics of Android slide in page transition. (flutter/engine#56762)
2024-11-25 chris@bracken.jp iOS: Eliminate logging of non-zero origin
platformviews (flutter/engine#56796)
2024-11-25 30870216+gaaclarke@users.noreply.github.com [impeller] gles:
started storing the number of handle deletions to avoid reallocation
(flutter/engine#56799)
2024-11-25 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from
9o0fWa2xVhmxV6Mtn... to 50xtjbMWWrqay_7m_... (flutter/engine#56795)
2024-11-25 skia-flutter-autoroll@skia.org Roll Dart SDK from
df716eaa6ed2 to 4b49546a1dfa (1 revision) (flutter/engine#56793)
2024-11-25 31859944+LongCatIsLooong@users.noreply.github.com [iOS] Full
keyboard access scrolling (flutter/engine#56606)
2024-11-25 jonahwilliams@google.com [android] remove fml_check from
surface_texture_external_texture (flutter/engine#56760)
2024-11-25 30870216+gaaclarke@users.noreply.github.com removed unused
variable for skia initialization (flutter/engine#56791)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from 9o0fWa2xVhmx to 50xtjbMWWrqa

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 zra@google.com 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
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
@gaaclarke
Started caching HandleGLES's hash and made them immutable (flutter/en…
6d9cdb5 
…gine#56800)

flutter#159177
test exempt: no functional change, performance

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@jonahwilliams jonahwilliams jonahwilliams approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

autosubmit Merge PR when tree becomes green via auto submit App e: impeller

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@gaaclarke @jonahwilliams