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

Issue 867652 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Sep 5
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

[Chameleon_HDMI] Chell - All Display tests are failing with Result of ###### -1920x1080: total 18##### wrong pixels (diff up to 116)

Project Member Reported by pgangishetty@chromium.org, Jul 25

Issue description

Since build version: R70-10906.0.0

DUT - Chell

All display tests except one fails with reason 'Result of ###### -1920x1080: total 18##### wrong pixels (diff up to 116)'


Failures screenshot:
https://screenshot.googleplex.com/cDcYw1KAtYz


 
Cc: waihong@chromium.org conradlo@chromium.org
ONLY display_SwitchMode.fast_switch test is still passing.

Affected tests are: 
display_HotPlugAtBoot.extended
display_HotPlugAtBoot.mirrored
display_HotPlugAtSuspend.extended
display_HotPlugAtSuspend.mirrored
display_LidCloseOpen.extended
display_Resolution.extended
display_Resolution.mirrored
display_SwitchMode

Dashboard view: https://screenshot.googleplex.com/6iiMTu6mPRy

pgangishetty@, can you test chell device with ext. monitor and try changing resolution?
220462296-chromeos-test%2Fchromeos2-row10-rack6-host5%2Fdisplay_LidCloseOpen.extended%2F021454-1920x1080-diff.png
4.5 KB View Download
Labels: chameleon-Fixit
+chameleon-Fixit label
Cc: marc...@chromium.org kbleicher@chromium.org
Components: OS>Kernel>Display
Labels: -Pri-2 -chameleon-Fixit Pri-1
All 5 hosts in lab are affected. Looks like a product issue.
Revoving chameleon-Fixit label.

Started with build M70-10906.0.0 / 70.0.3501.0 and the diffs to look for a change causing it are:
CrOs: https://crosland.corp.google.com/log/10905.0.0..10906.0.0
Chrome: https://chromium.googlesource.com/chromium/src/+log/70.0.3500.0..70.0.3501.0?n=10000

pshoma@, marcheu@, can you please check if any of the changes are to blame?


pgangishetty@, can you test two chell boards against same external display
- one device with M70-10906.0.0 / 70.0.3501.0
- one device with M70-10905.0.0 / 70.0.3500.0
and compare if there is a noticeable change?

Cc: -waihong@chromium.org tutankhamen@chromium.org dcasta...@chromium.org afakhry@chromium.org
+ some more folks


Re: #3 - Yes, I do see noticeable change on the external display when tested on:
- one device with M70-10906.0.0 / 70.0.3501.0 (img 10906_0_0.jpg)
- one device with M70-10905.0.0 / 70.0.3500.0 (img 10905_0_0.jpg)

Images present here: - https://pantheon.corp.google.com/storage/browser/chromiumos-test-logs/bugfiles/cr/867652
Cc: geohsu@chromium.org bhthompson@google.com
Labels: ReleaseBlock-Stable
Wow that's much different, no wonder tests are failing.
Thanks for testing manually and confirming. Nice pictures.
Could that be due to  issue 848540 ?
Owner: dcasta...@chromium.org
The revert https://chromium-review.googlesource.com/c/chromium/src/+/1146782 from  issue 848540  landed on chrome 70.0.3501.0, where the failure began.

+dcastagna@ as owner

Cc: dgagnon@chromium.org
Can anyone confirm this is now fixed and mark fixed if so?
The tests are passing now - https://screenshot.googleplex.com/wVK29DGHwzi - since M70 - 11017.0.0 / 70.0.3532.0 

Need merging to M69


Labels: M-69
Labels: Merge-Request-69
Labels: -Merge-Request-69 Merge-Approved-69
Merge approved, M69.
Tests are passing now on M69 since build 10895.47.0.
Screenshot: https://screenshot.googleplex.com/DKdhgkbfSHL
Status: Verified (was: Untriaged)
marked as verified.
Project Member

Comment 16 by sheriffbot@chromium.org, Sep 10

Cc: cindyb@chromium.org
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
Labels: -Merge-Approved-69

Sign in to add a comment