Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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] MediaSettings parameter for createCameraWithSettings #3586
[camera] MediaSettings parameter for createCameraWithSettings #3586
Changes from 91 commits
f294625
5ca79b9
e08e046
21617de
59d66d3
aeec92e
869b787
3cf6030
6ac763e
a40177a
fea1fc7
9f71f8d
4a0ac77
8fcf1ae
da4ecd8
16645ef
d7dea54
6caea7e
4bec8f3
11b43c1
772e011
dbb752d
7f01b93
3506af1
8f95dc2
6b1f96c
6f97b51
d96f9d5
401adce
8edd212
0a8fe17
d072dfb
accef7c
480761c
bdd74a8
ec5856f
6736802
9fdd1e0
8a55d87
7d88b16
86c95e9
c78cfc5
5597838
d006820
f60804d
d21dd5b
b08f5f1
d361800
a29b222
1020ee7
c31a502
dc0620e
d185de2
a1fe134
58aa61d
ecae539
2eb0657
a3f9c1d
0d3d0a9
076110f
ce05518
618a432
e81dfee
c95b6d5
e392d1d
72233fc
0a81ffe
8b0c6e8
dfe7b5a
277d4ae
863f79a
3bdf221
6060603
ca7e7c6
5146a65
4e881fd
4f07db2
a707e48
d50992d
fb99586
a0ca63a
59186e6
0653456
0cb06da
2cc95af
22656de
1401590
0ed67df
c24d21e
ff29e3f
fdd75fd
85a49f1
7088c8a
85d6f20
597dcc2
c1bac8b
8a90eee
d7bc0be
ea1777a
e0ffa44
f80e65f
8e14f0d
aa4802e
ea9e49e
aa6cc8b
8d5a701
97abc1b
2d7ec3d
def599e
0b5d0ac
d2dd528
14993b5
18be71c
b144466
c7140d5
81d3ba9
541b307
9776de3
a1bf2a0
25e2241
6fd3f07
8204c5c
ca0cc6b
cc6a0bd
5cf735d
59f3464
5d0d68c
14ce07f
d04448b
835ce37
556133e
ee7d582
0b5da81
b892a0e
5665dea
bed9858
4a6d0d7
6ff11f7
fb71469
d711155
1f92fe1
ad5acc3
b91875a
f278395
5be4485
672877f
21d838f
def04ca
3cfce50
0af2d84
05ce99c
8569ed4
dd03971
61d64ca
7ac9609
2639989
ccceded
86bfe6c
5cb9ef3
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: I don't think we necessarily need to deprecate this constructor. I don't see a reason we can't have both. This could still be used by those that don't want to mess with the bitrate, etc....
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does not appear to have been resolved.
(And if it is going to be deprecated, that needs to be done with the annotation, and mentioned in the release notes.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I remember correctly, you should be able to use the
this.withSettings
passthrough.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This hasn't been addressed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this a fix for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We use camera package intensively and there was error on closing widgets with mounted camera preview.
Actually
controller.value.isInitialized
is not enough, becauseconst CameraValue.uninitialized()
is never called. AndisInitialized
remainstrue
even on disposed controller.In scenarios with controller disposed in
didChangeAppLifecycleState
: theisPreviewPaused
is the only flag giving a clue to stop previewing and even listening to controller.value changes is not necessary if preview is not (already not) required.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I understand correctly, you added this to check if the controller is disposed. It is probably better to add a
isDisposed
method to theCameraController
instead. Although I'm not entirely sure why we added adebugCheckIsDisposed
instead.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bump on this discussion.