New issue
Advanced search Search tips

Issue 710230 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Blocking:
issue 618368



Sign in to add a comment

Merge surface_map_ and surface_view_map_ in GpuSurfaceTracker

Project Member Reported by liber...@chromium.org, Apr 10 2017

Issue description

|surface_map_| and |surface_view_map_| are (almost) parallel arrays on android.  refactor so that they're one map, and so that the Android surface isn't optional.
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d90f48c21a3dd8c93bbfb9dc53cdc4c644277702

commit d90f48c21a3dd8c93bbfb9dc53cdc4c644277702
Author: liberato <liberato@chromium.org>
Date: Mon May 01 23:44:50 2017

Update GpuSurfaceTracker to include Android surfaces.

Previously, GpuSurfaceTracker would add accelerated widgets and java
Surfaces via separate API calls.  Now, a java surface is required
when registering an accelerated widget (ANativeWindow) on Android.

BUG= 710230 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;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/2791723003
Cr-Commit-Position: refs/heads/master@{#468488}

[modify] https://crrev.com/d90f48c21a3dd8c93bbfb9dc53cdc4c644277702/chrome/browser/android/vr_shell/mailbox_to_surface_bridge.cc
[modify] https://crrev.com/d90f48c21a3dd8c93bbfb9dc53cdc4c644277702/content/browser/compositor/gpu_process_transport_factory.cc
[modify] https://crrev.com/d90f48c21a3dd8c93bbfb9dc53cdc4c644277702/content/browser/media/android/browser_surface_view_manager.cc
[modify] https://crrev.com/d90f48c21a3dd8c93bbfb9dc53cdc4c644277702/content/browser/renderer_host/compositor_impl_android.cc
[modify] https://crrev.com/d90f48c21a3dd8c93bbfb9dc53cdc4c644277702/gpu/ipc/common/gpu_surface_tracker.cc
[modify] https://crrev.com/d90f48c21a3dd8c93bbfb9dc53cdc4c644277702/gpu/ipc/common/gpu_surface_tracker.h
[modify] https://crrev.com/d90f48c21a3dd8c93bbfb9dc53cdc4c644277702/ui/compositor/test/in_process_context_factory.cc

Status: Fixed (was: Started)

Sign in to add a comment