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

Issue 12450016: Add purge all caches to our ashmem wrapper. (Closed)

Created:
7 years, 9 months ago by djsollen
Modified:
7 years, 9 months ago
Reviewers:
scroggo
CC:
skia-review_googlegroups.com
Visibility:
Public.

Description

Add purge all caches to our ashmem wrapper. THIS IS COMPLETELY UNTESTED Committed: http://code.google.com/p/skia/source/detail?r=8163

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M src/utils/android/ashmem.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/utils/android/ashmem.cpp View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
djsollen
7 years, 9 months ago (2013-03-14 15:59:43 UTC) #1
scroggo
On 2013/03/14 15:59:43, djsollen wrote: lgtm. Works for me in https://codereview.chromium.org/12433020/
7 years, 9 months ago (2013-03-14 17:36:09 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/djsollen@google.com/12450016/1
7 years, 9 months ago (2013-03-14 17:56:55 UTC) #3
commit-bot: I haz the power
Change committed as 8163
7 years, 9 months ago (2013-03-14 17:57:00 UTC) #4
djsollen
7 years, 9 months ago (2013-03-14 17:57:17 UTC) #5
Message was sent while issue was closed.
On 2013/03/14 17:57:00, I haz the power (commit-bot) wrote:
> Change committed as 8163

Thanks for testing it out!

Powered by Google App Engine
This is Rietveld 408576698