When maximum space size flags are set, overwrite the resource constraints.

When maximum space size flags are set, overwrite the resource constraints.

BUG=
R=rmcilroy@chromium.org

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

[rmcilroy, 2014-04-30 12:17:13]

https://codereview.chromium.org/262653002/diff/1/src/api.cc
File src/api.cc (right):

https://codereview.chromium.org/262653002/diff/1/src/api.cc#newcode474
src/api.cc:474: 
Would it be better to put this code in Heap::ConfigureHeap()?  That way the
flags would be respected even if the embedder passed their own custom created
ResourceConstraints via SetResourceConstraints?

[Hannes Payer (out of office), 2014-04-30 12:36:22]

https://codereview.chromium.org/262653002/diff/1/src/api.cc
File src/api.cc (right):

https://codereview.chromium.org/262653002/diff/1/src/api.cc#newcode474
src/api.cc:474: 
On 2014/04/30 12:17:13, rmcilroy wrote:
> Would it be better to put this code in Heap::ConfigureHeap()?  That way the
> flags would be respected even if the embedder passed their own custom created
> ResourceConstraints via SetResourceConstraints?

Yeah, makes sense. Done.

[rmcilroy, 2014-04-30 12:56:57]

lgtm - thanks!

[Hannes Payer (out of office), 2014-04-30 18:13:30]

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