Provide alternate constructor for specifying a simple thread's stacksize.

Provide alternate constructor for specifying a simple thread's stacksize.
For Native Client builds, make default stack size 2MB regardless of
newlib or glibc.

TEST=try bots
BUG=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=183618

[brettw, 2013-02-15 00:41:13]

Do you think it makes sense to pick a larger default stack size? If you were
hitting this, I bet other people will, too. And I bet most people could not
debug this kind of thing.

[nfullagar1, 2013-02-16 00:15:03]

It was a closer shave than originally thought.  Default stack size in NaCl for
newlib is 524288 bytes (512KB), and for glibc it should be significantly higher,
2MB.  Turns out the app in question was just hitting around 545128 bytes, so
just a little bit over newlib's default.  Ok, new upload will make NaCl always
use 2MB default size, regardless of newlib vs. glibc.  If a smaller or larger
stack size is desired the alternate constructor can be used.  Non-NaCl platforms
using the default constructor should have the same stack size as before this CL,
the default system stack size.

[brettw, 2013-02-16 04:19:40]

https://codereview.chromium.org/12259018/diff/8001/ppapi/utility/threading/si...
File ppapi/utility/threading/simple_thread.h (right):

https://codereview.chromium.org/12259018/diff/8001/ppapi/utility/threading/si...
ppapi/utility/threading/simple_thread.h:32: explicit SimpleThread(const
InstanceHandle& instance, const size_t stacksize);
I'd remove "const" for the argument here.

https://codereview.chromium.org/12259018/diff/8001/ppapi/utility/threading/si...
ppapi/utility/threading/simple_thread.h:61: SimpleThread(const SimpleThread&,
const size_t stacksize);
I'm remove "const" for the argument here.

[nfullagar1, 2013-02-19 18:59:10]

ptal, thx

https://codereview.chromium.org/12259018/diff/8001/ppapi/utility/threading/si...
File ppapi/utility/threading/simple_thread.h (right):

https://codereview.chromium.org/12259018/diff/8001/ppapi/utility/threading/si...
ppapi/utility/threading/simple_thread.h:32: explicit SimpleThread(const
InstanceHandle& instance, const size_t stacksize);
On 2013/02/16 04:19:40, brettw wrote:
> I'd remove "const" for the argument here.

Done.

https://codereview.chromium.org/12259018/diff/8001/ppapi/utility/threading/si...
ppapi/utility/threading/simple_thread.h:61: SimpleThread(const SimpleThread&,
const size_t stacksize);
On 2013/02/16 04:19:40, brettw wrote:
> I'm remove "const" for the argument here.

Done.

[brettw, 2013-02-20 19:05:51]

lgtm

[commit-bot: I haz the power, 2013-02-20 19:07:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nfullagar@chromium.org/12259018/12001

[commit-bot: I haz the power, 2013-02-20 21:15:02]

Change committed as 183618