Eagerly follow transitions to existing maps while json parsing.

Eagerly follow transitions to existing maps while json parsing.

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

[Toon Verwaest, 2012-10-16 17:38:13]

PTAL.

[danno, 2012-10-17 13:54:23]

https://chromiumcodereview.appspot.com/11184006/diff/2003/src/objects-inl.h
File src/objects-inl.h (right):

https://chromiumcodereview.appspot.com/11184006/diff/2003/src/objects-inl.h#n...
src/objects-inl.h:1418: int size =
As discussed, please convert this into something a little more readable.

https://chromiumcodereview.appspot.com/11184006/diff/2003/src/objects-inl.h#n...
src/objects-inl.h:1435: bool JSObject::TryTransitionToField(String* key) {
This should only be called by handlified code, it should be clear from API.

https://chromiumcodereview.appspot.com/11184006/diff/2003/src/objects.h
File src/objects.h (right):

https://chromiumcodereview.appspot.com/11184006/diff/2003/src/objects.h#newco...
src/objects.h:1626: inline bool TryTransitionToField(String* key);
must be static and use Handle<> just like other Handle APIs

[Toon Verwaest, 2012-10-17 14:03:40]

Addressed comments.

https://chromiumcodereview.appspot.com/11184006/diff/2003/src/objects-inl.h
File src/objects-inl.h (right):

https://chromiumcodereview.appspot.com/11184006/diff/2003/src/objects-inl.h#n...
src/objects-inl.h:1418: int size =
On 2012/10/17 13:54:23, danno wrote:
> As discussed, please convert this into something a little more readable.

Done.

https://chromiumcodereview.appspot.com/11184006/diff/2003/src/objects-inl.h#n...
src/objects-inl.h:1435: bool JSObject::TryTransitionToField(String* key) {
On 2012/10/17 13:54:23, danno wrote:
> This should only be called by handlified code, it should be clear from API.

Done.

https://chromiumcodereview.appspot.com/11184006/diff/2003/src/objects.h
File src/objects.h (right):

https://chromiumcodereview.appspot.com/11184006/diff/2003/src/objects.h#newco...
src/objects.h:1626: inline bool TryTransitionToField(String* key);
On 2012/10/17 13:54:23, danno wrote:
> must be static and use Handle<> just like other Handle APIs

Done.

[danno, 2012-10-17 14:07:49]

lgtm