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

Issue 1889703002: Android Java Surfaces can now be looked up via GpuSurfaceTracker (Closed)

Created:
4 years, 8 months ago by watk
Modified:
4 years, 8 months ago
Reviewers:
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

Android Java Surfaces can now be looked up via GpuSurfaceTracker AndroidVideoDecodeAccelerator needs to be able to retrieve registered Surfaces in order to decode to them. Previously this was only possible to do from the GPU process through SurfaceTextureManagerImpl, but now it can be done from the browser through GpuSurfaceTracker. This enables fullscreen video decoding with AVDA when in-process-gpu is enabled. BUG=601952 CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1874023002 Cr-Commit-Position: refs/heads/master@{#386531} (cherry picked from commit 2b597036f28ef872717bdfb0c337e43ae11170a7) Committed: https://chromium.googlesource.com/chromium/src/+/d4cfccda0714c51f307c0fca0f599b85c36ae9ce

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+37 lines, -15 lines) Patch
M content/browser/android/child_process_launcher_android.h View 1 chunk +2 lines, -0 lines 0 comments Download
M content/browser/android/child_process_launcher_android.cc View 1 chunk +7 lines, -0 lines 0 comments Download
M content/browser/gpu/gpu_surface_tracker.h View 1 chunk +4 lines, -0 lines 0 comments Download
M content/browser/gpu/gpu_surface_tracker.cc View 2 chunks +8 lines, -0 lines 0 comments Download
M content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java View 2 chunks +15 lines, -7 lines 0 comments Download
M gpu/ipc/common/gpu_surface_lookup.h View 1 chunk +1 line, -1 line 0 comments Download
M gpu/ipc/common/gpu_surface_lookup.cc View 1 chunk +0 lines, -7 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
watk
4 years, 8 months ago (2016-04-13 22:18:22 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
d4cfccda0714c51f307c0fca0f599b85c36ae9ce.

Powered by Google App Engine
This is Rietveld 408576698