New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 648528 link

Starred by 3 users

Regression: Black screen is seen while navigating to Camera

Project Member Reported by sandeepk...@techmahindra.com, Sep 20 2016

Issue description

Version: 55.0.2862.3/8818.0.0 (Official Build) dev-channel Daisy, Peppy
OS: Chrome

What steps will reproduce the problem?
(1) Sign in to user>> launch camera app>> click on toggle filters icon>>and observe

Expected: All the toggle filters should be loaded
Actual: Instead nothing is loading and it's showing different different colors

This is a Regression issue as it is working fine in M53: 53.0.2785.123/8530.87.0 (Official Build) stable-channel 

Note: Issue is not seen in Peppy device with today's build

Issue is not applicable to Linux and Windows
 
Actual.png
68.2 KB View Download
Issue is seen in Blaze and Mighty as well with today's build

Cc: songsuk@chromium.org pucchakayala@chromium.org
Able to reproduce the issue on Blaze using chrome version 55.0.2862.3/8818.0.0
Components: Blink>WebRTC>Video
Unable to do Webrtc call using https://apprtc.appspot.com/?debug=loopback in mighty,blaze and daisy devices with 55.0.2862.3/8818.0.0 dev.

Black blank screen is seen. No video is visible


Labels: -ReleaseBlock-Beta ReleaseBlock-Dev
Some other Repro steps:
1) Sign in to user>> go to Camera.app>> and observe

Expected: Preview should be shown
Actual: Instead black screen is seen

2)
1. Sign in to user>> go to settings>> go to people>> click on camera>> and observe

Expected: Preview should be shown
Actual: Instead black screen is seen
Summary: Regression: Black screen is seen while navigating to Camera (was: Regression: Toggle filters is not loading properly in Camera.app)
Components: Platform>Apps>Hangouts
Issue is seen in hangouts as well, while trying to have hangout in Daisy and Mighty devices (No video is seen)
Owner: wuchengli@chromium.org
I've asked Franky to bisect. I reproduced the black YouTube video on peach pi 8799.0.0 with tot chrome (synced today).
I could not reproduce this on elm 8799.0.0 with tot chrome (synced today).
Cc: dcasta...@chromium.org
Reverting https://codereview.chromium.org/2296433002 fixed the black video on peach pi (camera, apprtc, and youtube). I'll revert it.

https://codereview.chromium.org/2360453003 has been sent to CQ.
Issue is seen today's beta M53: 54.0.2840.33/8743.35.0 (Official Build) beta-channel  Blaze, Mighty and Daisy as well
How did that CL cause this breakage? "feature_flags_.chromium_image_ycbcr_420v = true;" ?
No idea. But "feature_flags_.chromium_image_ycbcr_420v = true;" only affects ARM. But #0 said Peppy (Intel) had the issue. I left the office and could not try reverting only chromium_image_ycbcr_420v.
Issue is not reproducible in Peppy device, have added it by mistake, but issue is still able to reproducible in Daisy, Blaze and Mighty devices with both the versions which are there in comment #0 and #11 
Labels: M-54 ReleaseBlock-Beta
Cc: shrawan@chromium.org
https://codereview.chromium.org/2360453003 was merged. I'm checking if the offending CL was in M54.
Confirmed the offending CL was merged to M54. https://codereview.chromium.org/2347453002
Labels: Merge-Request-54
I verified the revert fixed veyron minnie on M55 tot. 8809.0.0 on veyron minnie didn't have the issue and 8810.0.0 had the issue. The offending CL was picked up by 8810.0.0. I'm requesting a merge of https://codereview.chromium.org/2360453003 for M54.
Owner: dcasta...@chromium.org
Daniele. Can you merge the revert to M54 after it's approved? Thanks. I'm not sure if the revert has any dependency in M54.
It sounds like this bug is ARM specific based on the devices it is showing up in?
@#20, I'll take care of it, there isn't any dependency. I also have a fix that I'm going to land ASAP on ToT. Is merging the fix to 54 a possibility?

@#21, correct it's arm specific.
We were claiming that ycbcr_420v is supported on arm devices, and sometimes that's not the case.
SW decoded video streams in chromium would try to use that format and would fail. I have a fix already for that already.


Cc: hsiangc@chromium.org rohi...@chromium.org vsu...@chromium.org avkodipelli@chromium.org
Cc: josa...@google.com
Labels: -ReleaseBlock-Dev
+josafat for R54 merge.

Since this is already in the release, we should probably not block new dev pushes on it.

Comment 25 by josa...@google.com, Sep 22 2016

Labels: -Merge-Request-54 Merge-Approved-54
approved

Comment 26 by josa...@google.com, Sep 22 2016

@#22 Let's do revert for now and once fix is confirmed in ToT then you can request for merge again


Status: Fixed (was: Assigned)
Reverted here: https://codereview.chromium.org/2360323004
Cc: wuchengli@chromium.org
Issue 649136 has been merged into this issue.
Project Member

Comment 30 by sheriffbot@chromium.org, Sep 26 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 31 by josa...@google.com, Sep 26 2016

Is there any pending merge on this one ?
@josafat, not that I know. It has been reverted on ToT and 54 (https://codereview.chromium.org/2360323004)

I relanded the code, that should work now, on ToT but not in 54.
Labels: -Merge-Approved-54

Comment 34 by son...@google.com, Oct 7 2016

Status: Verified (was: Fixed)
Verified on M55 build 8872.0.0

Sign in to add a comment