Regression: Black patch on screenshot image is observed in 'Report an Issue' overlay.
Reported by
aiman.an...@etouch.net,
Apr 7 2017
|
||||
Issue descriptionChrome Version: 59.0.3065.0 (Official Build) (64-bit) (cohort: Stable)1d73b1aebe5f3a978a5a87c7502689b49ba1b134-refs/heads/master@{#462712}(32/64 Bit). OS: Windows(7,8,8.1,10) Steps to reproduce: 1.Launch chrome, Click on Wrench icon, go to help > Report an issue. 2.Observe the screenshot. Actual Result: Black patch is observed. Expected Result: Black patch should not be seen This is regression issue broken in 'M 59' and will soon update the bisect info: Manual Bisect Info: Good Build 59.0.3064.0 Bad Build 59.0.3065.0 Note: Will update MAC and LINUX OS status once build #59.0.3065.0 is available.
,
Apr 7 2017
I'm looking into the issue.
,
Apr 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/258527ddc93825a5feac1bd3de5f5efb8d975856 commit 258527ddc93825a5feac1bd3de5f5efb8d975856 Author: guidou <guidou@chromium.org> Date: Fri Apr 07 16:00:45 2017 Update constraints algorithm for video content capture. 1. Update default from 640x480 to 2880x1800. The goal is to use the native screen resolution as default, but this is an improvement over the current situation. 2. If a maximum resolution constraint is given, use that as default instead of the hardcoded default. This makes the algorithm more compatible with existing applications without breaking compatibility with the spec. BUG= 257097 , 709345 Review-Url: https://codereview.chromium.org/2806703002 Cr-Commit-Position: refs/heads/master@{#462876} [modify] https://crrev.com/258527ddc93825a5feac1bd3de5f5efb8d975856/content/renderer/media/media_stream_constraints_util_video_content.cc [modify] https://crrev.com/258527ddc93825a5feac1bd3de5f5efb8d975856/content/renderer/media/media_stream_constraints_util_video_content.h [modify] https://crrev.com/258527ddc93825a5feac1bd3de5f5efb8d975856/content/renderer/media/media_stream_constraints_util_video_content_unittest.cc
,
Apr 7 2017
,
Apr 7 2017
Patch crrev.com/462876 should fix it. rbasuvula: Can you verify?
,
Apr 8 2017
Issue 709392 has been merged into this issue.
,
Apr 10 2017
Issue 709867 has been merged into this issue.
,
Apr 11 2017
Tested the issue on Windows-10&7, Ubuntu 14.04 and Mac OS 10.12 using chrome latest Canary M59-59.0.3068.1(Mac & Linux) for windows checked in latest dev #59.0.3067.6 by following steps mentioned in the original comment. Observed that black patch is not observed in Mac and Linux but able to reproduce the issue in Windows 10&7. @guidou : Could you please take a look. Thank you!
,
Apr 11 2017
rbasuvula@: r463309 contains another patch that fixes other possible cases of black patches. 59.0.3068.1 (463474) has that patch, but 59.0.3067.6 (463157) does not. Try with 3068 on Windows when it becomes available. I believe it will be fixed there.
,
Apr 11 2017
Tested the issue on Windows-10, Ubuntu 14.04 and Mac OS 10.12 using chrome latest Canary M59-59.0.3068.1 by following steps mentioned in the original comment. Observed that Observed that screen is displaying as expected. Hence adding TE-Verified label. Please view the attachment for reference. Thank you! |
||||
►
Sign in to add a comment |
||||
Comment 1 by rbasuvula@chromium.org
, Apr 7 2017Labels: hasbisect-per-revision OS-Linux OS-Mac
Owner: guidou@chromium.org
Status: Assigned (was: Unconfirmed)