Vladimir E. Koltunov

Results 30 comments of Vladimir E. Koltunov

> Thanks for the contribution. This is very nice feature to have. Can you share a video capture after this change? You mean video recorded with this change? I recorded...

You can easily check stats including bitrate difference using `ffprobe`: This is info for first video: ![image](https://user-images.githubusercontent.com/8981380/229809745-43349f71-daec-47f8-a682-326eb975b4a6.png) This is info for second video: ![image](https://user-images.githubusercontent.com/8981380/229809247-2e962343-a2be-4e92-89e3-db4dcc939565.png)

@hellohuanlin > I saw a few update emails in the past few days. Feel free to re-request review (you can do so from the top right corner) I placed re-request...

@gmackall @ebjorklund01 PR is updated. The only test failure is about `federated_safety`. So, according to https://github.com/flutter/flutter/wiki/Contributing-to-Plugins-and-Packages#changing-federated-plugins, task 1 is done, and 2 is in review process.

@camsim99 https://github.com/flutter/packages/pull/3586#discussion_r1272548722 - Dropped.

@camsim99 https://github.com/flutter/packages/pull/3586#discussion_r1272553488 - refactored safety checks. Now preview can be rendered in paused but initialized state.

@camsim99 https://github.com/flutter/packages/pull/3586#discussion_r1293672956 > Does this require a downgrade? Sorry, I must've thought you were upgrading the version here. updated to 7.3.0

@stuartmorgan I have force pushed #3615, following https://github.com/flutter/flutter/wiki/Contributing-to-Plugins-and-Packages#changing-federated-plugins. Can you review #3615?

camera-platform-interface changes approved (federated-safety related). @stuartmorgan / @camsim99, can you please make secondary review? (suggested by @bparrishMines in https://github.com/flutter/packages/pull/3615#pullrequestreview-1629389267) Also @ditman requested some changes, applied.