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

Issue 667993002: small tweaks to SkResourceCache::Key (Closed)

Created:
6 years, 2 months ago by mtklein_C
Modified:
6 years, 2 months ago
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Project:
skia
Visibility:
Public.

Description

small tweaks to SkResourceCache::Key - remove unused method and obsolete note - add a reminder that yes, we are checking counts are equal BUG=skia: Committed: https://skia.googlesource.com/skia/+/484bbe5be63d6c516e8b190634117989de0f9a26

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -3 lines) Patch
M src/core/SkResourceCache.h View 2 chunks +1 line, -3 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
mtklein
Just noticed these while looking at Florin's CL.
6 years, 2 months ago (2014-10-21 17:37:11 UTC) #2
reed1
lgtm
6 years, 2 months ago (2014-10-21 17:42:31 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/667993002/1
6 years, 2 months ago (2014-10-21 17:48:56 UTC) #5
commit-bot: I haz the power
6 years, 2 months ago (2014-10-21 17:55:31 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) as 484bbe5be63d6c516e8b190634117989de0f9a26

Powered by Google App Engine
This is Rietveld 408576698