Map::Normalize() introduced as single entry point for map normalization and Map::NotifyLeafMapLayou…

Map::Normalize() introduced as single entry point for map normalization and Map::NotifyLeafMapLayoutChange() made private.

R=verwaest@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=21117

[Igor Sheludko, 2014-04-30 13:36:45]

PTAL

[Igor Sheludko, 2014-04-30 15:50:23]

Addressed offline comments.

[Toon Verwaest, 2014-05-01 11:24:34]

lgtm with comment

https://codereview.chromium.org/263663002/diff/40001/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/263663002/diff/40001/src/objects.cc#newcode7222
src/objects.cc:7222: new_map->EquivalentToForNormalization(*fast_map, mode)) {
Seems like EquivalentToForNormalization should be moved into
cache->Get(fast_map, mode), since hash collisions are an internal property of
the cache.

[Igor Sheludko, 2014-05-02 09:47:38]

https://codereview.chromium.org/263663002/diff/40001/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/263663002/diff/40001/src/objects.cc#newcode7222
src/objects.cc:7222: new_map->EquivalentToForNormalization(*fast_map, mode)) {
On 2014/05/01 11:24:34, Toon Verwaest wrote:
> Seems like EquivalentToForNormalization should be moved into
> cache->Get(fast_map, mode), since hash collisions are an internal property of
> the cache.

Done.

[Igor Sheludko, 2014-05-02 10:27:20]

Committed patchset #3 manually as r21117 (presubmit successful).