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

With Camera Open & URL reload, Android UI Fails to show in chrome bro… #19

Open
wants to merge 1 commit into
base: r/ics3a/main
Choose a base branch
from

Conversation

Sapna1-singh
Copy link

…wser

Whenever camera is opened and URL is reloaded, Camera close will be called from framework and On the other side Streamer will send REQ_CAP to the camera-vhal.

Before receiving the camera close from camera-vhal,streamer is sending the REQ_CAP and by this time, camera state is still in camera-opened state which inturn ignores this Capability request as camera is in opened state yet.

Handle the REQ_CAP packet irrespective of the camera state.

Tracked-On: OAM-108772

Whenever camera is opened and URL is reloaded, camera close will be
called from framework and on the other side streamer will send
capability request to the camera-vhal. camera vhal ignores the
capability request as it is received when the camera is in opened
state. Streamer is stuck waiting for the response to capability
request which results in no UI streamed out.

Fix the issue by handling capability request even when camera 
is in opened state in camera vhal.

Tracked-On: OAM-108772
Signed-off-by: Singh, Sapna1 <[email protected]>
@Sapna1-singh Sapna1-singh force-pushed the ww15_camerabugfix branch 3 times, most recently from 343ef40 to 71f626a Compare April 14, 2023 18:08
@sysopenci sysopenci added the Stale Stale label for inactive open prs label Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale Stale label for inactive open prs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants