Break cyclic reference between utils and platform.

Break cyclic reference between utils and platform.

Platform includes utils.h for the definition of Vector<>, so utils.h
can't include platform.h and has to use memcpy instead of OS::MemCopy.

We split out Vector<> into its own header to break this cycle
dependency.

BUG=none
R=mstarzinger@chromium.org
LOG=n

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

[Michael Starzinger, 2014-04-25 08:28:25]

LGTM.

https://codereview.chromium.org/251753002/diff/1/src/libplatform/default-plat...
File src/libplatform/default-platform.h (right):

https://codereview.chromium.org/251753002/diff/1/src/libplatform/default-plat...
src/libplatform/default-platform.h:61: static const int kMaxThreadPoolSize;
Just out of curiosity: What is the reason to move this into the .cc file?

[jochen (gone - plz use gerrit), 2014-04-25 08:36:42]

https://codereview.chromium.org/251753002/diff/1/src/libplatform/default-plat...
File src/libplatform/default-platform.h (right):

https://codereview.chromium.org/251753002/diff/1/src/libplatform/default-plat...
src/libplatform/default-platform.h:61: static const int kMaxThreadPoolSize;
On 2014/04/25 08:28:25, Michael Starzinger wrote:
> Just out of curiosity: What is the reason to move this into the .cc file?

it didn't link otherwise :-/

[jochen (gone - plz use gerrit), 2014-04-25 08:37:33]

Committed patchset #1 manually as r20962 (presubmit successful).

[ulan, 2014-04-25 11:35:24]

On 2014/04/25 08:37:33, jochen wrote:
> Committed patchset #1 manually as r20962 (presubmit successful).

This broke fast/js/JSON-stringify.html layout test on linux64:

Threw exception RangeError: Maximum call stack size exceeded