Bug 1024265 - Part 0: Fix comments. r=bholley DONTBUILD

--HG--
extra : rebase_source : 8bc1cf12182f241f0a1cfdaf27d54c9c56e69bda
This commit is contained in:
David Major 2014-06-13 10:30:33 +12:00
parent 525203360e
commit dee69d7ce7

View File

@ -3142,8 +3142,8 @@ XPCJSRuntime::XPCJSRuntime(nsXPConnect* aXPConnect)
const size_t kSystemCodeBuffer = 10 * 1024;
// Our "default" stack is what we use in configurations where we don't have
// a compelling reason to do things differently. This is effectively 1MB on
// 32-bit platforms and 2MB on 64-bit platforms.
// a compelling reason to do things differently. This is effectively 512KB
// on 32-bit platforms and 1MB on 64-bit platforms.
const size_t kDefaultStackQuota = 128 * sizeof(size_t) * 1024;
// Set stack sizes for different configurations. It's probably not great for
@ -3157,7 +3157,7 @@ XPCJSRuntime::XPCJSRuntime(nsXPConnect* aXPConnect)
const size_t kTrustedScriptBuffer = 180 * 1024;
#elif defined(MOZ_ASAN)
// ASan requires more stack space due to red-zones, so give it double the
// default (2MB on 32-bit, 4MB on 64-bit). ASAN stack frame measurements
// default (1MB on 32-bit, 2MB on 64-bit). ASAN stack frame measurements
// were not taken at the time of this writing, so we hazard a guess that
// ASAN builds have roughly thrice the stack overhead as normal builds.
// On normal builds, the largest stack frame size we might encounter is
@ -3165,8 +3165,7 @@ XPCJSRuntime::XPCJSRuntime(nsXPConnect* aXPConnect)
const size_t kStackQuota = 2 * kDefaultStackQuota;
const size_t kTrustedScriptBuffer = 246 * 1024;
#elif defined(XP_WIN)
// 1MB is the default stack size on Windows, so the default 1MB stack quota
// we'd get on win32 is slightly too large. Use 900k instead. And since
// 1MB is the default stack size on Windows, so use 900k. And since
// windows stack frames are 3.4k each, let's use a buffer of 50k.
const size_t kStackQuota = 900 * 1024;
const size_t kTrustedScriptBuffer = 50 * 1024;