Issue 2250353002: Introduce copyToVector for HeapHashCountedSet

Issue 2250353002: Introduce copyToVector for HeapHashCountedSet (Closed)

Created:
4 years, 4 months ago by yhirano

Modified:
4 years, 4 months ago

Reviewers:
haraken

CC:
chromium-reviews, oilpan-reviews, Mads Ager (chromium), blink-reviews, blink-reviews-wtf_chromium.org, kouhei+heap_chromium.org, Mikhail

Base URL:
https://chromium.googlesource.com/chromium/src.git@remove-some-hash-counted-set-copy-to-vector

Target Ref:
refs/pending/heads/master

Project:
chromium

Visibility:
Public.

Introduce copyToVector for HeapHashCountedSet BUG=587663 Committed: https://crrev.com/2e4771df2490ad8b8d3b53afb39887490ca8833a Cr-Commit-Position: refs/heads/master@{#412499}

Created: 4 years, 4 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+47 lines, -7 lines)			Patch
M	third_party/WebKit/Source/platform/heap/HeapAllocator.h	View	1 chunk	+6 lines, -0 lines	0 comments	Download
M	third_party/WebKit/Source/platform/heap/HeapTest.cpp	View	2 chunks	+36 lines, -0 lines	0 comments	Download
M	third_party/WebKit/Source/wtf/HashCountedSet.h	View	1 chunk	+5 lines, -7 lines	0 comments	Download

Depends on Patchset:

Dependent Patchsets:

Total messages: 14 (9 generated)

yhirano

The function is planned to be used at https://codereview.chromium.org/2191633003/.

4 years, 4 months ago (2016-08-17 08:21:55 UTC) #5

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/2250353002/1

4 years, 4 months ago (2016-08-17 11:37:17 UTC) #10

commit-bot: I haz the power

4 years, 4 months ago (2016-08-17 11:47:40 UTC) #14

Message was sent while issue was closed.

Patchset 1 (id:??) landed as
https://crrev.com/2e4771df2490ad8b8d3b53afb39887490ca8833a
Cr-Commit-Position: refs/heads/master@{#412499}