Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(112)

Issue 1963773003: Merge to M51: Various fixes to prevent playback hang on MotoX. (Closed)

Created:
4 years, 7 months ago by DaleCurtis
Modified:
4 years, 7 months ago
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@2704
Target Ref:
refs/pending/branch-heads/2704
Project:
chromium
Visibility:
Public.

Description

Merge to M51: Various fixes to prevent playback hang on MotoX. e886ba2 Avoid waiting on the SurfaceTexture until we need to. cecc8a4 Increase cases where back/front buffering early rendering occur. 9b07026 Queue more input buffers and process more output if possible. 9771835 Avoid unnecessary post task during frame delivery. fb00bd3 ReleaseOutputBuffer to surface back and front buffers where possible. 6c5e1df Delay actual flush and release in AVDA 4ddcd18 Store AVDACodecImage list in shared state, cleanup callers. BUG=598963, 601066 TBR=liberato, timav (cherry picked from commit 5cda4bb2b9d1a7b7821f87df58337928361ce2c3) Committed: https://chromium.googlesource.com/chromium/src/+/b6b2141b0683cad4d1f7ccfcaa6350c31437cdf2

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+628 lines, -228 lines) Patch
M content/common/gpu/media/android_copying_backing_strategy.h View 1 chunk +7 lines, -7 lines 0 comments Download
M content/common/gpu/media/android_copying_backing_strategy.cc View 2 chunks +2 lines, -3 lines 0 comments Download
M content/common/gpu/media/android_deferred_rendering_backing_strategy.h View 4 chunks +20 lines, -10 lines 0 comments Download
M content/common/gpu/media/android_deferred_rendering_backing_strategy.cc View 9 chunks +89 lines, -42 lines 0 comments Download
M content/common/gpu/media/android_video_decode_accelerator.h View 5 chunks +40 lines, -7 lines 0 comments Download
M content/common/gpu/media/android_video_decode_accelerator.cc View 18 chunks +169 lines, -60 lines 0 comments Download
M content/common/gpu/media/avda_codec_image.h View 6 chunks +59 lines, -23 lines 0 comments Download
M content/common/gpu/media/avda_codec_image.cc View 8 chunks +99 lines, -57 lines 0 comments Download
M content/common/gpu/media/avda_shared_state.h View 4 chunks +48 lines, -4 lines 0 comments Download
M content/common/gpu/media/avda_shared_state.cc View 3 chunks +61 lines, -3 lines 0 comments Download
M content/common/gpu/media/gpu_video_decode_accelerator.cc View 1 chunk +5 lines, -3 lines 0 comments Download
M media/base/video_decoder.h View 2 chunks +8 lines, -3 lines 0 comments Download
M media/filters/ffmpeg_video_decoder.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/filters/gpu_video_decoder.cc View 3 chunks +19 lines, -4 lines 0 comments Download
M media/filters/vpx_video_decoder.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (1 generated)
DaleCurtis
4 years, 7 months ago (2016-05-10 04:15:43 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
b6b2141b0683cad4d1f7ccfcaa6350c31437cdf2.

Powered by Google App Engine
This is Rietveld 408576698