Issue 126823003: posix LaunchProcess: remove more iterator usage that was missed in r243401

Issue 126823003: posix LaunchProcess: remove more iterator usage that was missed in r243401 (Closed)

Created:
6 years, 11 months ago by mattm

Modified:
6 years, 11 months ago

Reviewers:
sehr, willchan no longer on Chromium, Paweł Hajdan Jr.

CC:
chromium-reviews, erikwright+watch_chromium.org

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

posix LaunchProcess: remove more iterator usage that was missed in r243401 BUG=331459 TBR=sehr@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=243720

Created: 6 years, 11 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+15 lines, -17 lines)			Patch
M	base/process/launch.h	View	2 chunks	+1 line, -2 lines	0 comments	Download
M	base/process/launch_posix.cc	View	5 chunks	+12 lines, -13 lines	0 comments	Download
M	components/nacl/zygote/nacl_fork_delegate_linux.cc	View	1 chunk	+2 lines, -2 lines	0 comments	Download

Total messages: 7 (0 generated)

willchan no longer on Chromium

Lgtm PS I think Brad left awhile back so you probably want a different reviewer. ...

6 years, 11 months ago (2014-01-08 22:13:10 UTC) #4

mattm

On 2014/01/08 22:13:10, willchan wrote: > Lgtm > > PS I think Brad left awhile ...

6 years, 11 months ago (2014-01-08 22:19:55 UTC) #5

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mattm@chromium.org/126823003/1

6 years, 11 months ago (2014-01-08 22:24:14 UTC) #6

Message was sent while issue was closed.

Change committed as 243720