Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2978)

Issue 131680043: Add 'GetAllocationSize' to HeapInterface. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 7 months ago by chrisha
Modified:
9 years, 7 months ago
CC:
sawbuck-changes_googlegroups.com
Base URL:
http://sawbuck.googlecode.com/svn/trunk
Visibility:
Public.

Description

Add 'GetAllocationSize' to HeapInterface. Currently no heaps support this feature. It will be added to the existing heaps one by one in later CLs. BUG= R=sebmarchand@chromium.org Committed: https://code.google.com/p/sawbuck/source/detail?r=2282

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -0 lines) Patch
M syzygy/agent/asan/heap.h View 2 chunks +16 lines, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/ctmalloc_heap.h View 1 chunk +1 line, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/ctmalloc_heap.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/internal_heap.h View 1 chunk +1 line, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/internal_heap.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/large_block_heap.h View 1 chunk +1 line, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/large_block_heap.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/simple_block_heap.h View 1 chunk +1 line, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/simple_block_heap.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/win_heap.h View 1 chunk +1 line, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/win_heap.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/zebra_block_heap.h View 1 chunk +1 line, -0 lines 0 comments Download
M syzygy/agent/asan/heaps/zebra_block_heap.cc View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 4
chrisha
PTAL?
9 years, 7 months ago (2014-08-29 14:56:25 UTC) #1
Sébastien Marchand
lgtm.
9 years, 7 months ago (2014-08-29 14:58:18 UTC) #2
chrisha
Thanks, committing!
9 years, 7 months ago (2014-08-29 15:13:08 UTC) #3
chrisha
9 years, 7 months ago (2014-08-29 15:20:08 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as 2282.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b