`new Intl.Segmenter().segment()` causes SEGFAULT with `--with-intl=small-icu` and no runtime ICU data about node

[targos]

Backtrace:

#0  0x0000fffff68cba28 in v8::internal::JSSegments::Create(v8::internal::Isolate*, v8::internal::Handle<v8::internal::JSSegmenter>, v8::internal::Handle<v8::internal::String>) () from /lib64/libnode.so.115
#1  0x0000fffff64bfee0 [PAC] in v8::internal::Builtin_SegmenterPrototypeSegment(int, unsigned long*, v8::internal::Isolate*) ()
   from /lib64/libnode.so.115
#2  0x0000fffff6305964 [PAC] in Builtins_CEntry_Return1_ArgvOnStack_BuiltinExit () from /lib64/libnode.so.115

I don't know if this is a V8 bug or if we don't initialize it correctly.

/cc @nodejs/i18n-api

[richardlau]

FWIW we strip out ICU break iterator data with small-icu

node/tools/icu/icu_small.json

Line 19 in bf39716

"brkitr": "none",

node/tools/icu/icu_small.json

Lines 34 to 40 in bf39716

	"remove": [
	"cnvalias.icu",
	"postalCodeData.res",
	"genderList.res",
	"brkitr/root.res",
	"unames.icu"
	],

Also https://bugs.chromium.org/p/v8/issues/detail?id=3345 referenced

node/tools/icu/icu-generic.gyp

Lines 38 to 41 in bf39716

	# TODO(srl295): reenable following pending
	# https://code.google.com/p/v8/issues/detail?id=3345
	# (saves some space)
	'UCONFIG_NO_BREAK_ITERATION=0',

cc @srl295

[srl295]

FWIW we strip out ICU break iterator data with small-icu

node/tools/icu/icu_small.json

Line 19 in bf39716

"brkitr": "none",

node/tools/icu/icu_small.json

Lines 34 to 40 in bf39716

	"remove": [
	"cnvalias.icu",
	"postalCodeData.res",
	"genderList.res",
	"brkitr/root.res",
	"unames.icu"
	],

Also https://bugs.chromium.org/p/v8/issues/detail?id=3345 referenced

node/tools/icu/icu-generic.gyp

Lines 38 to 41 in bf39716

	# TODO(srl295): reenable following pending
	# https://code.google.com/p/v8/issues/detail?id=3345
	# (saves some space)
	'UCONFIG_NO_BREAK_ITERATION=0',

cc @srl295

In 2014 there wasn't an Intl.Segmenter.. probably these should be included.

However, v8 shouldn't crash, that should be fixed in v8.

We do have some code (IIRC) that detects small-icu and monkeypatched Intl with a warning about small icu.