Issue 966243002: Reduce platform/heap/ wtf includes.

Issue 966243002: Reduce platform/heap/ wtf includes. (Closed)

Created:
5 years, 9 months ago by sof

Modified:
5 years, 9 months ago

Reviewers:
oilpan-reviews, tkent, haraken

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

Base URL:
svn://svn.chromium.org/blink/trunk

Target Ref:
refs/heads/master

Project:
blink

Visibility:
Public.

More Reviews

Description

Reduce platform/heap/ wtf includes. Rely on wtf/Forward.h where possible, reducing CPP's #include count ever so slightly when processing platform/heap/Handle.h R= BUG=420515 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=191082

Patch Set 1 #

Created: 5 years, 9 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+7 lines, -20 lines)			Patch
M	Source/platform/heap/Heap.h	View	1 chunk	+1 line, -3 lines	0 comments	Download
M	Source/platform/heap/ThreadState.h	View	1 chunk	+1 line, -3 lines	0 comments	Download
M	Source/platform/heap/Visitor.h	View	3 chunks	+3 lines, -14 lines	0 comments	Download
M	Source/wtf/Forward.h	View	2 chunks	+2 lines, -0 lines	0 comments	Download

Messages

Total messages: 7 (2 generated)

Expand Messages | Collapse Messages | Show Generated Messages | Hide Generated Messages

sof

Please take a look. Every little bit helps...hope it's ok not to split out the ...

5 years, 9 months ago (2015-03-02 06:07:09 UTC) #2

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/966243002/1

5 years, 9 months ago (2015-03-02 06:31:12 UTC) #5

commit-bot: I haz the power

5 years, 9 months ago (2015-03-02 06:33:55 UTC) #7

Message was sent while issue was closed.

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=191082

Expand Messages | Collapse Messages | Show Generated Messages | Hide Generated Messages