Split struct should not be readonly

[stephentoub]

Doing so prohibits desirable optimizations, like making the TokenString property lazily initialized from the original string (otherwise, we end up allocating a new string for every split even if the string isn't used).

[stephentoub]

cc: @tarekgh

[stephentoub]

addressed in #7012