Issue 2954563002: cros: Fix wallpaper manager indexing vector out-of-range issue

Issue 2954563002: cros: Fix wallpaper manager indexing vector out-of-range issue (Closed)

Created:
3 years, 6 months ago by Qiang(Joe) Xu

Modified:
3 years, 6 months ago

Reviewers:
xdai1

CC:
chromium-reviews, alemate+watch_chromium.org, achuith+watch_chromium.org, oshima+watch_chromium.org, davemoore+watch_chromium.org

Target Ref:
refs/heads/master

Project:
chromium

Visibility:
Public.

cros: Fix wallpaper manager indexing vector out-of-range issue BUG=none TBR=xdai@chromium.org TEST=chrome doesn't crash Review-Url: https://codereview.chromium.org/2954563002 Cr-Commit-Position: refs/heads/master@{#481703} Committed: https://chromium.googlesource.com/chromium/src/+/fc185a85198149b326df53956fe8e30401215d3a

Created: 3 years, 6 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+1 line, -1 line)			Patch
	M	chrome/browser/chromeos/login/users/wallpaper/wallpaper_manager.cc	View		1 chunk	+1 line, -1 line	0 comments	Download

Total messages: 8 (4 generated)

commit-bot: I haz the power

CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2954563002/1

3 years, 6 months ago (2017-06-22 22:14:30 UTC) #2

Qiang(Joe) Xu

Sorry for this -- I TBRed you as it is quite obvious.

3 years, 6 months ago (2017-06-22 22:14:53 UTC) #3

xdai1

On 2017/06/22 22:14:53, Qiang(Joe) Xu wrote: > Sorry for this -- I TBRed you as ...

3 years, 6 months ago (2017-06-22 22:18:14 UTC) #5

commit-bot: I haz the power

3 years, 6 months ago (2017-06-22 22:59:01 UTC) #8

Message was sent while issue was closed.

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/fc185a85198149b326df53956fe8...