These two pieces of info should be redundant, so we should just use FunctionBox::extent.

Outside of powering BytecodeSection's source notes generation, the line and
column numbers were almost entirely unused, except to power a single assertion
that lineno and column agreed between the stencil (BCE derived) and script.

However, by switching to using the SharedContext extents, we lose the ability
to always assert they're the same, because in the old system dynamically
allocated functions used externally provided column info that disagrees with
the parser result. In this patch the script still gets the same column
number as before, but we can't assert that the value is the same as what you
could get out of the BCE, because the BCE number is actually more accurate
than the script.

Fixing the script column should be followup.

Depends on D70410

Backed out 2 changesets (Bug 1627373) for causing wpt and mochitest-chrome failures

Push with failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=297655223&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&revision=17dadcf8617fba804cf7f01d5747ae586526b624

WPT failures
Failure log 1: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=297655223&repo=autoland&lineNumber=3684
Failure log 2: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=297656547&repo=autoland&lineNumber=1984
Failure log 3: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=297658778&repo=autoland&lineNumber=9640

Mochitest-chrome failures
Failure log 4: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=297657400&repo=autoland&lineNumber=141251
Failure log 5: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=297660215&repo=autoland&lineNumber=2283
Failure log 6: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=297657950&repo=autoland&lineNumber=10306

Backout link: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=297657950&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&revision=811772d60380a95ab43581973435b872f32c7ad0

Based on the failures, it looks like something is wrong with how Modules are handled.

Correct; This gave modules a line number of 1, ignoring the line and column of the options.

Corrected I think the remaining oranges on Try are intermittents:https://treeherder.mozilla.org/#/jobs?repo=try&revision=afcdd3de02e413d138386509fa5bb2d0875bd383

https://hg.mozilla.org/mozilla-central/rev/8e1c3f7241e4
https://hg.mozilla.org/mozilla-central/rev/f3d35a537298