New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 869507 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Aug 2
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

SurfaceManager::GetLatestInFlightSurface should be able to return the primary

Project Member Reported by akaba@google.com, Jul 31

Issue description

GetLatestInFlightSurface should return the primary if it exists and active.  There is no reason for it not to do so and this will make working with it easier.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 2

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

commit 995425d1170e9a8ebbdceafbc60961bcf5e99a54
Author: akaba <akaba@google.com>
Date: Thu Aug 02 20:48:08 2018

SurfaceManager::GetLatestInFlightSurface should be able to return the primary

GetLatestInFlightSurface should return the primary if it exists and active.
There is no reason for it not to do so and this will make working with
it easier.

Bug:  869507 
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
Change-Id: Icae9da8527de69555e8c925740c08b187f827b02
Reviewed-on: https://chromium-review.googlesource.com/1157140
Commit-Queue: Andre Kaba <akaba@google.com>
Commit-Queue: Fady Samuel <fsamuel@chromium.org>
Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580311}
[modify] https://crrev.com/995425d1170e9a8ebbdceafbc60961bcf5e99a54/components/viz/service/display/surface_aggregator.cc
[modify] https://crrev.com/995425d1170e9a8ebbdceafbc60961bcf5e99a54/components/viz/service/frame_sinks/surface_synchronization_unittest.cc
[modify] https://crrev.com/995425d1170e9a8ebbdceafbc60961bcf5e99a54/components/viz/service/surfaces/surface.cc
[modify] https://crrev.com/995425d1170e9a8ebbdceafbc60961bcf5e99a54/components/viz/service/surfaces/surface_manager.cc

Status: Fixed (was: Assigned)

Sign in to add a comment