Issue metadata
Sign in to add a comment
|
Regression: Black screen is seen while navigating to Camera |
||||||||||||||||||||||
Issue descriptionVersion: 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
,
Sep 20 2016
Able to reproduce the issue on Blaze using chrome version 55.0.2862.3/8818.0.0
,
Sep 20 2016
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
,
Sep 20 2016
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
,
Sep 20 2016
,
Sep 20 2016
Issue is seen in hangouts as well, while trying to have hangout in Daisy and Mighty devices (No video is seen)
,
Sep 21 2016
I've asked Franky to bisect. I reproduced the black YouTube video on peach pi 8799.0.0 with tot chrome (synced today).
,
Sep 21 2016
I could not reproduce this on elm 8799.0.0 with tot chrome (synced today).
,
Sep 21 2016
Reverting https://codereview.chromium.org/2296433002 fixed the black video on peach pi (camera, apprtc, and youtube). I'll revert it.
,
Sep 21 2016
https://codereview.chromium.org/2360453003 has been sent to CQ.
,
Sep 21 2016
Issue is seen today's beta M53: 54.0.2840.33/8743.35.0 (Official Build) beta-channel Blaze, Mighty and Daisy as well
,
Sep 21 2016
How did that CL cause this breakage? "feature_flags_.chromium_image_ycbcr_420v = true;" ?
,
Sep 21 2016
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.
,
Sep 21 2016
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
,
Sep 21 2016
,
Sep 21 2016
,
Sep 22 2016
https://codereview.chromium.org/2360453003 was merged. I'm checking if the offending CL was in M54.
,
Sep 22 2016
Confirmed the offending CL was merged to M54. https://codereview.chromium.org/2347453002
,
Sep 22 2016
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.
,
Sep 22 2016
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.
,
Sep 22 2016
It sounds like this bug is ARM specific based on the devices it is showing up in?
,
Sep 22 2016
@#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.
,
Sep 22 2016
,
Sep 22 2016
+josafat for R54 merge. Since this is already in the release, we should probably not block new dev pushes on it.
,
Sep 22 2016
approved
,
Sep 22 2016
@#22 Let's do revert for now and once fix is confirmed in ToT then you can request for merge again
,
Sep 22 2016
,
Sep 23 2016
,
Sep 23 2016
Issue 649136 has been merged into this issue.
,
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
,
Sep 26 2016
Is there any pending merge on this one ?
,
Sep 27 2016
@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.
,
Sep 29 2016
,
Oct 7 2016
Verified on M55 build 8872.0.0 |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by sandeepk...@techmahindra.com
, Sep 20 2016