1485510 - Fix mochitests that don't work with CM jsterm enabled

Status: RESOLVED FIXED

(DevTools :: Console, enhancement, P1)

Description

[Brian Grinstead [:bgrins]]

As I discovered in https://bugzilla.mozilla.org/show_bug.cgi?id=1463669#c11, after browser_console_consolejsm_output.js happens we never actually run any of the tests with the new jsterm (even when the pref is set).

Fixing this surfaces a number of test failures, like: https://treeherder.mozilla.org/#/jobs?repo=try&revision=f696488a3bb5cbd49ae3f8cc6e61124e920e03c6&selectedJob=195363439. Each of which can be confirmed in a local build by running the test individually. I'm planning to turn off the new frontend for those tests (since it's not running in them anyway), and we can use this bug to fix them.

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1485510 - Fix old-jsterm UI glitches; r=bgrins.

The completion node was wrongly positionned as soon as the input overflowed.
We fix this by setting the completionNode height in resizeInput,
like we do for the inputNode.

The autocompletionPopup was a bit off due to 2 things:
- in the function that was returning the chevronWidth, we
were substracting the autocomplete popup padding. But the
autocomplete popup already handles that itself.
- in the function that was computing the character width,
we were using offsetWidth which returned a rounded value.
This means that the further the autocompletion was displayed
at, the more the popup would be off. We use getBoundingClientRect().width
instead which gives us a decimal value.

And we also make sure to not alter the scrolling position in the inputNode
when accepting an autocompletion result (a test is added).

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1485510 - Fix mochitests that don't work with CM jsterm enabled; r=bgrins.

Comment 3

[Nicolas Chevobbe [:nchevobbe]]

TRY push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=74e6e36fb78a0fedc1305c403fd56c75319a6d97

I fixed the eslint failure in the last phabricator review

Comment 4

[Brian Grinstead [:bgrins]]

Comment on attachment 9003771 [details]
Bug 1485510 - Fix mochitests that don't work with CM jsterm enabled; r=bgrins.

Brian Grinstead [:bgrins] has approved the revision.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b6494cda84fc
Fix mochitests that don't work with CM jsterm enabled; r=bgrins.

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/b6494cda84fc

Comment 7

[Brian Grinstead [:bgrins]]

Attachment: overflow-autocomplete.gif

Should this positioning be fixed by the patch? I'm seeing it both with and without the patch applied.

Comment 8

[Brian Grinstead [:bgrins]]

(In reply to Brian Grinstead [:bgrins] from comment #7)
> Should this positioning be fixed by the patch? I'm seeing it both with and
> without the patch applied.

Also note that this may well have been an existing issue - I'm pretty sure the old jsterm never handled autocomplete on multiline statements well. We don't need to spend a lot of time fixing that if this is the case.

Comment 9

[Brian Grinstead [:bgrins]]

Those last two comments were intended for the "Fix old-jsterm UI glitches" patch which is linked up with this bug. Probably that patch should be attached to a different bug?