New issue
Advanced search Search tips

Issue 762615 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Clean up in layer.cc

Project Member Reported by wutao@chromium.org, Sep 6 2017

Issue description

1. use range-based loop to enumerate |children_|
2. NULL->nullptr


 
I will check the issue. Thanks.
Submitted https://chromium-review.googlesource.com/c/chromium/src/+/691535 for your consideration. Thanks.
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 3 2017

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

commit de445af03efead537cd6715f5e32586bf1968868
Author: nikhil <nikhil.sahni@samsung.com>
Date: Tue Oct 03 14:38:42 2017

Clean up in layer.cc

Using range-based loop to enumerate
|children_| and changing null to
nullptr.

BUG= 762615 


Change-Id: Ie0246f5bbb3a3d4d8cb857ca4719e4e94a7a4252
Reviewed-on: https://chromium-review.googlesource.com/691535
Reviewed-by: David Reveman <reveman@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Reviewed-by: Valery Arkhangorodsky <varkha@chromium.org>
Commit-Queue: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506034}
[modify] https://crrev.com/de445af03efead537cd6715f5e32586bf1968868/ui/compositor/layer.cc

Comment 4 by wutao@chromium.org, Oct 10 2017

Status: Fixed (was: Available)
Thanks, Nikhil.

Sign in to add a comment