Added an Isolate* field to NoTrackDoubleFieldsForSerializerScope, PlatformFeatureScope and BinaryOp…

Added an Isolate* field to NoTrackDoubleFieldsForSerializerScope, PlatformFeatureScope and BinaryOpIC::State.

The serializer state and even the CPU features will be per-Isolate
later. Currently we get away with global state, because mksnapshot
runs single-threaded and has only 1 Isolate, but this will change.
Furthermore, these changes are yet another prerequisite for removing a
catch-22 at initialization time when we try to enable serialization.

This CL is similar in spirit to r20919, BTW.

BUG=359977
LOG=y
R=mstarzinger@chromium.org

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

[Sven Panne, 2014-04-24 13:10:49]

Yet another spin-off from a bigger CL to get the startup right...

[Michael Starzinger, 2014-04-25 08:03:29]

LGTM.

https://codereview.chromium.org/250553005/diff/1/src/ic.h
File src/ic.h (right):

https://codereview.chromium.org/250553005/diff/1/src/ic.h#newcode746
src/ic.h:746: explicit State(Isolate* isolate, ExtraICState extra_ic_state);
nit: No longer needs to be "explicit" here.

[Sven Panne, 2014-04-25 08:27:08]

Feedback addressed, landing...

https://codereview.chromium.org/250553005/diff/1/src/ic.h
File src/ic.h (right):

https://codereview.chromium.org/250553005/diff/1/src/ic.h#newcode746
src/ic.h:746: explicit State(Isolate* isolate, ExtraICState extra_ic_state);
On 2014/04/25 08:03:30, Michael Starzinger wrote:
> nit: No longer needs to be "explicit" here.

Done.

[Sven Panne, 2014-04-25 08:40:41]

Committed patchset #2 manually as r20963 (presubmit successful).