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

Issue 876347 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

LocalSurfaceId::ToString should use %u instead of %d

Project Member Reported by akaba@chromium.org, Aug 21

Issue description

LocalSurfaceId::ToString should us %u to print parent/child sequence numbers since they are unsigned integers.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 21

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

commit 13c54c9e44a0abd8596da31490d9455c86bf04f0
Author: akaba <akaba@google.com>
Date: Tue Aug 21 18:54:02 2018

LocalSurfaceId::ToString should use %u instead of %d

LocalSurfaceId::ToString should us %u to print
parent/child sequence numbers since they are unsigned integers.

Bug:  876347 
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
Change-Id: Icac3ff853e97b9d8951490fdf7509dfad7449cf7
Reviewed-on: https://chromium-review.googlesource.com/1183785
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@{#584865}
[modify] https://crrev.com/13c54c9e44a0abd8596da31490d9455c86bf04f0/components/viz/common/surfaces/local_surface_id.cc

Status: Fixed (was: Assigned)

Sign in to add a comment