Issue metadata
Sign in to add a comment
|
Video playback (HTML5) stops/breaks. Produces glitches, black boxes, flickering on OSX 10.9.5 / Mavericks
Reported by
sebastia...@gmail.com,
Jul 27 2016
|
||||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_5) AppleWebKit/601.7.7 (KHTML, like Gecko) Version/9.1.2 Safari/537.86.7 Example URL: Steps to reproduce the problem: 1. Go to youtube or netflix 2. play any video 3. wait for some time (~5 to 15min possible!) What is the expected behavior? Normal video playback. What went wrong? Video playback (HTML5) stops/breaks. Will produces glitches, black boxes, flickering. Did this work before? Yes Chrome Version 51.0.2704.106 Is it a problem with Flash or HTML5? HTML5 Does this work in other browsers? Yes Chrome version: 52.0.2743.85 Channel: stable OS Version: OS X 10.9.5 Flash Version: Shockwave Flash 22.0 r0 No problems on OSX 10.11 El Capitan. This was introduces with recent Chrome update to Version 52.0.2743.85. No problems before. Tested it with old version. Works fine again. (disabled autoupdate).
,
Jul 28 2016
Yes! Definitively seems related to https://bugs.chromium.org/p/chromium/issues/detail?id=631485. Thanks! Will stick with 51.x for now. Can one merge this issue into the other one?
,
Jul 28 2016
,
Aug 1 2016
Issue 611310 has been merged into this issue.
,
Aug 1 2016
Un-merging this, since it is a unique bug. The relevant context is that the issue goes away when we flush the AVSampleBufferDisplayLayer before each call to enqueue a new sample Description of fix: - https://bugs.chromium.org/p/chromium/issues/detail?id=631485#c54 Confirmation that it works: - https://bugs.chromium.org/p/chromium/issues/detail?id=631485#c55 This has been fixed in M52 by disabling AVSampleBufferDisplayLayer (sadness). I'll fix up the flushing behavior and merge it into M53.
,
Aug 5 2016
Moving this nonessential bug to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Aug 20 2016
Requesting merge to M53. The fix for this was made for M52 in https://bugs.chromium.org/p/chromium/issues/detail?id=631485#c58, but needs to be merged and restricted to 10.9
,
Aug 20 2016
Adding RBS 53, because I forgot the merge
,
Aug 20 2016
Approving merge to M53 branch 2785 for cl listed at #7. Please merge before Monday, 5:00 PM PT so we can take it for next week M53 LAST beta release. Thank you.
,
Aug 22 2016
A friendly reminder that M53 Stable is launching VERY soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP (before 5:00 PM PT, Tuesday) so we can take it for this week LAST Beta release for Desktop. Thank you! Note: Merge has to happen by Friday, August 26th, 5:00 PM PST in order to make into the desktop Stable final build cut.
,
Aug 22 2016
Oops, I landed https://codereview.chromium.org/2269473002/ against issue 631485. This is good to go for M53. Needs a proper blacklist for M54.
,
Aug 22 2016
Thank you ccameron@. Removing "Merge-Approved-53" and applying "merge-merged-2785" label per comment #11.
,
Aug 23 2016
,
Aug 24 2016
->54
,
Sep 8 2016
Issue 638215 has been merged into this issue.
,
Sep 15 2016
Do we need a merge to M54 branch?
,
Sep 21 2016
Yes, I need to do this!
,
Sep 23 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f33642d15eb090f26cf37c40b04f2ba62a95e243 commit f33642d15eb090f26cf37c40b04f2ba62a95e243 Author: ccameron <ccameron@chromium.org> Date: Fri Sep 23 02:09:54 2016 Mac video: Add blacklist entry for AVSampleBufferDisplayLayer This leaks IOSurfaces on 10.9. Include 10.10 in the blacklist because it has not been sufficiently tested. BUG= 632178 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2366433002 Cr-Commit-Position: refs/heads/master@{#420543} [modify] https://crrev.com/f33642d15eb090f26cf37c40b04f2ba62a95e243/gpu/config/gpu_driver_bug_list_json.cc [modify] https://crrev.com/f33642d15eb090f26cf37c40b04f2ba62a95e243/gpu/config/gpu_driver_bug_workaround_type.h [modify] https://crrev.com/f33642d15eb090f26cf37c40b04f2ba62a95e243/gpu/ipc/service/image_transport_surface_overlay_mac.mm [modify] https://crrev.com/f33642d15eb090f26cf37c40b04f2ba62a95e243/ui/accelerated_widget_mac/ca_layer_tree_coordinator.h [modify] https://crrev.com/f33642d15eb090f26cf37c40b04f2ba62a95e243/ui/accelerated_widget_mac/ca_layer_tree_coordinator.mm [modify] https://crrev.com/f33642d15eb090f26cf37c40b04f2ba62a95e243/ui/accelerated_widget_mac/ca_layer_tree_unittest_mac.mm [modify] https://crrev.com/f33642d15eb090f26cf37c40b04f2ba62a95e243/ui/accelerated_widget_mac/ca_renderer_layer_tree.h [modify] https://crrev.com/f33642d15eb090f26cf37c40b04f2ba62a95e243/ui/accelerated_widget_mac/ca_renderer_layer_tree.mm
,
Sep 26 2016
Currently 10.9.5 / Mavericks version is not available with Chrome-TE HYD team, So requesting MTV TE team to look in to this issue for verification.
,
Sep 27 2016
The CL is already merged to M53, so please request a merge to M54 ASAP. We are planning to cut the RC cut @ 4.00PM PST,09/27(Tuesday).
,
Sep 27 2016
,
Sep 27 2016
[Automated comment] Request affecting a post-stable build (M53), manual review required.
,
Sep 27 2016
Whoops, meant M54
,
Sep 27 2016
Your change meets the bar and is auto-approved for M54 (branch: 2840)
,
Sep 27 2016
,
Sep 27 2016
,
Sep 27 2016
,
Sep 28 2016
Rechecked this on chrome version 54.0.2840.41 on MAC 10.11.6. Played some videos in Netflix and on youtube. Videos with length more than 20 to 25 minutes were played. No glitches, black boxes or flickering was observed. Adding TE-Verified labels.
,
Sep 28 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/39c18b9eb123aa7e478249c5f71a4259285bcac8 commit 39c18b9eb123aa7e478249c5f71a4259285bcac8 Author: Christopher Cameron <ccameron@chromium.org> Date: Tue Sep 27 21:31:27 2016 Mac video: Add blacklist entry for AVSampleBufferDisplayLayer This leaks IOSurfaces on 10.9. Include 10.10 in the blacklist because it has not been sufficiently tested. BUG= 632178 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2366433002 Cr-Commit-Position: refs/heads/master@{#420543} (cherry picked from commit f33642d15eb090f26cf37c40b04f2ba62a95e243) Review URL: https://codereview.chromium.org/2372343002 . Cr-Commit-Position: refs/branch-heads/2840@{#555} Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607} [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/gpu/config/gpu_driver_bug_list_json.cc [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/gpu/config/gpu_driver_bug_workaround_type.h [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/gpu/ipc/service/image_transport_surface_overlay_mac.mm [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/ui/accelerated_widget_mac/ca_layer_tree_coordinator.h [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/ui/accelerated_widget_mac/ca_layer_tree_coordinator.mm [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/ui/accelerated_widget_mac/ca_layer_tree_unittest_mac.mm [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/ui/accelerated_widget_mac/ca_renderer_layer_tree.h [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/ui/accelerated_widget_mac/ca_renderer_layer_tree.mm
,
Sep 28 2016
,
Sep 28 2016
Issue 632741 has been merged into this issue.
,
Sep 30 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b6a6d7c3849e22f549937a42ad4a0177c06fa1df commit b6a6d7c3849e22f549937a42ad4a0177c06fa1df Author: ccameron <ccameron@chromium.org> Date: Fri Sep 30 18:58:45 2016 color: Set GpuMemoryBuffer color spaces for Canvas Change Canvas2DLayerBridge::ImageInfo to hold the gfx::GpuMemoryBuffer, instead of going through the glCreateGpuMemoryBufferImageCHROMIUM function. This GL function will be deleted in the future. Call gfx::GpuMemoryBuffer::SetColorSpaceForScanout, which inform CoreAnimation of the IOSurface's color space. Add a gfx::ColorSpace::FromSkColorSpace stub. BUG= 632178 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2364633003 Cr-Commit-Position: refs/heads/master@{#422167} [modify] https://crrev.com/b6a6d7c3849e22f549937a42ad4a0177c06fa1df/content/renderer/renderer_blink_platform_impl.cc [modify] https://crrev.com/b6a6d7c3849e22f549937a42ad4a0177c06fa1df/content/renderer/renderer_blink_platform_impl.h [modify] https://crrev.com/b6a6d7c3849e22f549937a42ad4a0177c06fa1df/gpu/ipc/common/surface_handle.h [modify] https://crrev.com/b6a6d7c3849e22f549937a42ad4a0177c06fa1df/third_party/WebKit/Source/platform/BUILD.gn [modify] https://crrev.com/b6a6d7c3849e22f549937a42ad4a0177c06fa1df/third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridge.cpp [modify] https://crrev.com/b6a6d7c3849e22f549937a42ad4a0177c06fa1df/third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridge.h [modify] https://crrev.com/b6a6d7c3849e22f549937a42ad4a0177c06fa1df/third_party/WebKit/Source/platform/graphics/DEPS [modify] https://crrev.com/b6a6d7c3849e22f549937a42ad4a0177c06fa1df/third_party/WebKit/public/platform/Platform.h [modify] https://crrev.com/b6a6d7c3849e22f549937a42ad4a0177c06fa1df/ui/gfx/color_space.cc [modify] https://crrev.com/b6a6d7c3849e22f549937a42ad4a0177c06fa1df/ui/gfx/color_space.h [modify] https://crrev.com/b6a6d7c3849e22f549937a42ad4a0177c06fa1df/ui/gfx/native_widget_types.h
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/39c18b9eb123aa7e478249c5f71a4259285bcac8 commit 39c18b9eb123aa7e478249c5f71a4259285bcac8 Author: Christopher Cameron <ccameron@chromium.org> Date: Tue Sep 27 21:31:27 2016 Mac video: Add blacklist entry for AVSampleBufferDisplayLayer This leaks IOSurfaces on 10.9. Include 10.10 in the blacklist because it has not been sufficiently tested. BUG= 632178 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2366433002 Cr-Commit-Position: refs/heads/master@{#420543} (cherry picked from commit f33642d15eb090f26cf37c40b04f2ba62a95e243) Review URL: https://codereview.chromium.org/2372343002 . Cr-Commit-Position: refs/branch-heads/2840@{#555} Cr-Branched-From: 1ae106dbab4bddd85132d5b75c670794311f4c57-refs/heads/master@{#414607} [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/gpu/config/gpu_driver_bug_list_json.cc [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/gpu/config/gpu_driver_bug_workaround_type.h [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/gpu/ipc/service/image_transport_surface_overlay_mac.mm [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/ui/accelerated_widget_mac/ca_layer_tree_coordinator.h [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/ui/accelerated_widget_mac/ca_layer_tree_coordinator.mm [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/ui/accelerated_widget_mac/ca_layer_tree_unittest_mac.mm [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/ui/accelerated_widget_mac/ca_renderer_layer_tree.h [modify] https://crrev.com/39c18b9eb123aa7e478249c5f71a4259285bcac8/ui/accelerated_widget_mac/ca_renderer_layer_tree.mm |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by redins...@gmail.com
, Jul 27 2016