Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(178)

Issue 15981008: skia/ext: Updated lazy_pixel_ref_utils style. (Closed)

Created:
7 years, 6 months ago by vmpstr
Modified:
7 years, 6 months ago
Reviewers:
Tom Hudson, tomhudson
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

skia/ext: Updated lazy_pixel_ref_utils style. This patch makes lazy_pixel_ref_utils* style consistent with Chromium style. BUG=245787 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=205376

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+50 lines, -56 lines) Patch
M skia/ext/lazy_pixel_ref_utils.h View 1 chunk +1 line, -1 line 0 comments Download
M skia/ext/lazy_pixel_ref_utils.cc View 16 chunks +49 lines, -55 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
vmpstr
PTAL.
7 years, 6 months ago (2013-06-10 16:53:23 UTC) #1
Tom Hudson
lgtm
7 years, 6 months ago (2013-06-10 16:55:46 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vmpstr@chromium.org/15981008/1
7 years, 6 months ago (2013-06-10 16:56:00 UTC) #3
commit-bot: I haz the power
Retried try job too often on chromium_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_presubmit&number=7986
7 years, 6 months ago (2013-06-10 17:18:47 UTC) #4
tomhudson
lgtm The other me says LGTM too.
7 years, 6 months ago (2013-06-10 17:33:43 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vmpstr@chromium.org/15981008/1
7 years, 6 months ago (2013-06-10 17:33:50 UTC) #6
vmpstr
On 2013/06/10 17:33:43, tomhudson wrote: > lgtm > > The other me says LGTM too. ...
7 years, 6 months ago (2013-06-10 17:45:58 UTC) #7
commit-bot: I haz the power
7 years, 6 months ago (2013-06-11 01:41:18 UTC) #8
Message was sent while issue was closed.
Change committed as 205376

Powered by Google App Engine
This is Rietveld 408576698