New issue
Advanced search Search tips

Issue 609359 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

gfx::Range::ToString() works incorrectly.

Project Member Reported by karandeepb@chromium.org, May 5 2016

Issue description

Version: 50.0.2661.94 
OS: All

gfx::Range::InvalidRange().ToString() prints {-1,-1} when it should print {4294967295,4294967295}.

 
Status: Fixed (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, May 6 2016

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

commit 212a05f54124d15d7678afee40f91219fc90717f
Author: karandeepb <karandeepb@chromium.org>
Date: Fri May 06 01:56:47 2016

Fix gfx::Range::ToString() to use the correct format specifier.

This bug seems to have been introduced in crrev.com/1671403002. Also added a
test which fails on the current master.

BUG= 609359 

Review-Url: https://codereview.chromium.org/1948283002
Cr-Commit-Position: refs/heads/master@{#391986}

[modify] https://crrev.com/212a05f54124d15d7678afee40f91219fc90717f/ui/gfx/range/range.cc
[modify] https://crrev.com/212a05f54124d15d7678afee40f91219fc90717f/ui/gfx/range/range_unittest.cc

Sign in to add a comment