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

Issue 800472 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

HostFrameSinkManager: Store FrameSink debug label

Project Member Reported by fsamuel@google.com, Jan 9 2018

Issue description

The FrameSink debug label is useful to look up in the host process and should repush that state when the viz process crashes.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 16 2018

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

commit 544d25463645648ae53393dd3fdd772414eac214
Author: Fady Samuel <fsamuel@chromium.org>
Date: Tue Jan 16 16:47:38 2018

FrameSink Debug label: store in HostFrameSinkManager

The FrameSink debug label is likely useful in the host process (for
HitTestQuery for example) and should be re-sent to the new Viz instance
if the viz process crashes.

Bug:  800472 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: Ida613d7e555fcf0b4419665150b1fbb7909f4c36
Reviewed-on: https://chromium-review.googlesource.com/857597
Reviewed-by: kylechar <kylechar@chromium.org>
Commit-Queue: Fady Samuel <fsamuel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#529443}
[modify] https://crrev.com/544d25463645648ae53393dd3fdd772414eac214/components/viz/host/host_frame_sink_manager.cc
[modify] https://crrev.com/544d25463645648ae53393dd3fdd772414eac214/components/viz/host/host_frame_sink_manager.h
[modify] https://crrev.com/544d25463645648ae53393dd3fdd772414eac214/components/viz/service/surfaces/surface_manager.cc

Status: Fixed (was: Started)

Sign in to add a comment