Move static flag modifications to V8::InitializeOncePerProcessImpl().

Move static flag modifications to V8::InitializeOncePerProcessImpl().

BUG=
R=mstarzinger@chromium.org

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

[Michael Starzinger, 2013-06-17 15:17:25]

https://codereview.chromium.org/17250006/diff/1/src/v8.cc
File src/v8.cc (right):

https://codereview.chromium.org/17250006/diff/1/src/v8.cc#newcode274
src/v8.cc:274: if (FLAG_sweeper_threads == 0) {
This should actually say "FLAG_sweeper_threads <= 0" because otherwise
--sweeper-threads=-1 will crash.

https://codereview.chromium.org/17250006/diff/1/src/v8.cc#newcode289
src/v8.cc:289: if (FLAG_parallel_marking && FLAG_marking_threads == 0) {
This will start marking threads with "--noparallel-marking
--marking-threads=100". Also "--marking-threads=-1" will also lead to crashes I
think.

[Hannes Payer (out of office), 2013-06-17 17:07:30]

Now the flag definition is stricter than before, i.e., the actual enabling flags
are more powerful than the number of threads flags.

https://codereview.chromium.org/17250006/diff/1/src/v8.cc
File src/v8.cc (right):

https://codereview.chromium.org/17250006/diff/1/src/v8.cc#newcode274
src/v8.cc:274: if (FLAG_sweeper_threads == 0) {
On 2013/06/17 15:17:25, Michael Starzinger wrote:
> This should actually say "FLAG_sweeper_threads <= 0" because otherwise
> --sweeper-threads=-1 will crash.

Done.

https://codereview.chromium.org/17250006/diff/1/src/v8.cc#newcode289
src/v8.cc:289: if (FLAG_parallel_marking && FLAG_marking_threads == 0) {
On 2013/06/17 15:17:25, Michael Starzinger wrote:
> This will start marking threads with "--noparallel-marking
> --marking-threads=100". Also "--marking-threads=-1" will also lead to crashes
I
> think.

Done.

[Michael Starzinger, 2013-06-17 17:12:52]

LGTM.

[Hannes Payer (out of office), 2013-06-19 08:57:32]

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