Off by one(1) error in column addressing: fixing `originalPositionFor()` API for indexed maps + added test adapted from the 0.6.x dev tree#399

Closed

GerHobbelt wants to merge 8 commits intomozilla:masterfrom

GerHobbelt:patch-8

Commits on Jun 18, 2019

fix crash in url util function due to undefined root value: `sourceRoot` is checked everywhere using `!= null` except in 1(one) spot, which caused a crash in my test set, while debugging indexed ma…
GerHobbelt
committed
when `sourceContentFor(aSource, nullOnMissing=false)` shouldn't this situation also throw an exception? In the other situation, where this API might also produce a NULL return, an exception *is* th…
GerHobbelt
committed
fixing `originalPositionFor()` API for indexed maps + added test adapted from the 0.6.x dev tree: it turned out that due an off-by-1 error in the column value, half of the mapping nodes weren't hit…
GerHobbelt
committed
Merge branch 'patch-3' into patch-8
GerHobbelt
committed
allow the sourceMapGenerator code to also enjoy the `nullOrMissing` argument/option for the `consumer.sourceContentFor()` API.
GerHobbelt
committed
also use the `nullOrNothing` option/argument in the lib/source-node.js code to restore intended behaviour. All tests pass.
GerHobbelt
committed
Merge branch 'patch-7' into patch-8
GerHobbelt
committed
remove console diagnostics from the tests. All tests pass, once again.
GerHobbelt
committed