Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[camera_windows] Revert: Support image streams on Windows platform #7951

Merged

Conversation

stuartmorgan
Copy link
Contributor

This reverts commit b31a279 (#7067)

The implementation had significant bugs:

  • It had asynchronous results, but never responded to them, violating the engine API contract and preventing futures from completing.
  • It std::moved pointers between classes, while still keeping them as ivars and global references and referring to them from long-lived handlers.

Pre-launch Checklist

…lutter#7067)"

This reverts commit b31a279.

The implementation had significant bugs:
- It had asynchronous results, but never responded to them, violating
  the engine API contract and preventing futures from completing.
- It `std::move`d pointers between classes, while still keeping them as
  ivars and referring to them from long-lived handlers.
@stuartmorgan
Copy link
Contributor Author

This is straight git revert other than the CHANGELOG and version.

Copy link
Member

@cbracken cbracken left a comment

Choose a reason for hiding this comment

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

LGTM stamp from a Japanese personal seal

@stuartmorgan stuartmorgan added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 4, 2024
@auto-submit auto-submit bot merged commit b3a5e33 into flutter:main Nov 4, 2024
76 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 4, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 4, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 4, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 5, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 5, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Nov 5, 2024
flutter/packages@796afa3...7219431

2024-11-05 [email protected] [vector_graphics*] Relax dependency constraints of vector_graphics, vector_graphics_codec, vector_graphics_compiler, flutter_svg  (flutter/packages#8018)
2024-11-04 [email protected] [various] Add `missing_code_block_language_in_doc_comment` lint to flutter/packages. (flutter/packages#6473)
2024-11-04 [email protected] [various] Update example apps to Kotlin 1.9.0 (flutter/packages#7998)
2024-11-04 [email protected] [go_router] add current state getter (flutter/packages#7651)
2024-11-04 [email protected] Applied Gradle Plugins Declaratively for Multiple Plugin Example Apps (flutter/packages#7968)
2024-11-04 [email protected] Roll Flutter from f86b777 to 8591d0c (16 revisions) (flutter/packages#8015)
2024-11-04 [email protected] [camera_windows] Revert: Support image streams on Windows platform (flutter/packages#7951)
2024-11-02 [email protected] [camera] Use Pigeon for Windows C++->Dart (flutter/packages#8001)
2024-11-02 [email protected] [script/tool] update dependencies (flutter/packages#7992)
2024-11-01 [email protected] Roll Flutter from 0fe6153 to f86b777 (16 revisions) (flutter/packages#8000)
2024-11-01 [email protected] [path_parsing] deprecate utility functions that should be private (flutter/packages#7993)

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-flutter-autoroll
Please CC [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
liff added a commit to CodemateLtd/packages that referenced this pull request Nov 26, 2024
liff added a commit to CodemateLtd/packages that referenced this pull request Dec 5, 2024
liff added a commit to CodemateLtd/packages that referenced this pull request Dec 5, 2024
liff added a commit to CodemateLtd/packages that referenced this pull request Jan 15, 2025
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 p: camera platform-windows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants