Bug 1064634 - Use the same media thread stack size on all platforms where we specify a size. Also document the rationale behind the MOZ_ASAN exception. r=cpearce

This commit is contained in:
Matthew Gregan 2014-09-30 15:30:58 +13:00
parent f9b69fd80a
commit ba679130a1

View File

@ -168,9 +168,11 @@ static const int32_t MAX_VIDEO_HEIGHT = 3000;
void ScaleDisplayByAspectRatio(nsIntSize& aDisplay, float aAspectRatio);
// The amount of virtual memory reserved for thread stacks.
#if (defined(XP_WIN) || defined(LINUX)) && !defined(MOZ_ASAN)
#define MEDIA_THREAD_STACK_SIZE (128 * 1024)
#elif defined(XP_MACOSX) && !defined(MOZ_ASAN)
#if defined(MOZ_ASAN)
// Use the system default in ASAN builds, because the default is assumed to be
// larger than the size we want to use and is hopefully sufficient for ASAN.
#define MEDIA_THREAD_STACK_SIZE nsIThreadManager::DEFAULT_STACK_SIZE
#elif defined(XP_WIN) || defined(XP_MACOSX) || defined(LINUX)
#define MEDIA_THREAD_STACK_SIZE (256 * 1024)
#else
// All other platforms use their system defaults.