[heap] Free list refactoring of finding nodes.

[heap] Free list refactoring of finding nodes.

R=ulan@chromium.org
BUG=chromium:524425
LOG=N

Committed: https://crrev.com/deccbde1614c5e1696d9166b4b353fc108e745c1
Cr-Commit-Position: refs/heads/master@{#31183}

[Michael Lippautz, 2015-10-08 14:45:00]

mlippautz@chromium.org changed reviewers:
+ ulan@chromium.org

[Michael Lippautz, 2015-10-08 14:45:01]

Carving down the bigger (and more involved) CL of dividing memory upon
compaction tasks. This should be a non-risky change.

PTAL

https://codereview.chromium.org/1394863002/diff/1/src/heap/spaces.cc
File src/heap/spaces.cc (right):

https://codereview.chromium.org/1394863002/diff/1/src/heap/spaces.cc#newcode2321
src/heap/spaces.cc:2321: FreeSpace* FreeList::FindNodeIn(FreeListCategoryType
category, int* node_size) {
FYI: This one will be used in a follow-up CL for finding memory , similar to
  if (node == nullptr) node = FindNodeIn(kHuge, &node_size);
  if (node == nullptr) node = FindNodeIn(kLarge, &node_size);
  if (node == nullptr) node = FindNodeIn(kMedium, &node_size);
  ...

[ulan, 2015-10-08 14:51:25]

lgtm

[Michael Lippautz, 2015-10-08 14:53:30]

The CQ bit was checked by mlippautz@chromium.org

[commit-bot: I haz the power, 2015-10-08 14:53:40]

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

[commit-bot: I haz the power, 2015-10-08 15:17:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-08 15:17:31]

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