Skip to content

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

Merged
auto-submit[bot] merged 2 commits into
flutter:mainfrom
stuartmorgan-g:camera-windows-revert-streaming
Nov 4, 2024
Merged

[camera_windows] Revert: Support image streams on Windows platform#7951
auto-submit[bot] merged 2 commits into
flutter:mainfrom
stuartmorgan-g:camera-windows-revert-streaming

Conversation

@stuartmorgan-g

Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
CollaboratorAuthor

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

@cbrackencbracken left a comment

Copy link
Copy Markdown
Member

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-gstuartmorgan-g added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 4, 2024
@auto-submit
auto-submitBot merged commit b3a5e33 into flutter:mainNov 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 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-submitBot pushed a commit to flutter/flutter that referenced this pull request Nov 5, 2024
flutter/packages@796afa3...7219431
2024-11-05 jonahwilliams@google.com [vector_graphics*] Relax dependency constraints of vector_graphics, vector_graphics_codec, vector_graphics_compiler, flutter_svg (flutter/packages#8018)
2024-11-04 kallentu@google.com [various] Add `missing_code_block_language_in_doc_comment` lint to flutter/packages. (flutter/packages#6473)
2024-11-04 stuartmorgan@google.com [various] Update example apps to Kotlin 1.9.0 (flutter/packages#7998)
2024-11-04 cedvandenbosch@gmail.com [go_router] add current state getter (flutter/packages#7651)
2024-11-04 jessiewong401@gmail.com Applied Gradle Plugins Declaratively for Multiple Plugin Example Apps (flutter/packages#7968)
2024-11-04 engine-flutter-autoroll@skia.org Roll Flutter from f86b777 to 8591d0c (16 revisions) (flutter/packages#8015)
2024-11-04 stuartmorgan@google.com [camera_windows] Revert: Support image streams on Windows platform (flutter/packages#7951)
2024-11-02 stuartmorgan@google.com [camera] Use Pigeon for Windows C++->Dart (flutter/packages#8001)
2024-11-02 kevmoo@users.noreply.github.com [script/tool] update dependencies (flutter/packages#7992)
2024-11-01 engine-flutter-autoroll@skia.org Roll Flutter from 0fe6153 to f86b777 (16 revisions) (flutter/packages#8000)
2024-11-01 kevmoo@users.noreply.github.com [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 flutter-ecosystem@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
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 Feb 11, 2025
liff added a commit to CodemateLtd/packages that referenced this pull request Feb 20, 2025
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
…lutter#7951)
This reverts commit b31a279 (flutter#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::move`d pointers between classes, while still keeping them as ivars and global references and referring to them from long-lived handlers.
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
…lutter#7951)
This reverts commit 9727278 (flutter#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::move`d pointers between classes, while still keeping them as ivars and global references and referring to them from long-lived handlers.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmitMerge PR when tree becomes green via auto submit Appp: cameraplatform-windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@stuartmorgan-g@cbracken