@thehogfather, the changes look good to me. Initialize the uninitialized property with an empty object and get rid of the undefined/null error message down the road.

Is _codeMirror._lineFolds where you are storing all of the fold state information for each instance of CodeMirror opened in brackets?

The only problem I see is the Travis build is failing. @ficristo or @petetnt any ideas what might be causing the build fail?

@lkcampbell

Is _codeMirror._lineFolds where you are storing all of the fold state information for each instance of CodeMirror opened in brackets?

Yes that is correct.

With respect to Travis failing, I'll pull upstream and push to trigger the build again. The error messages show it did not get very far into the build process.

That did no seem to help. Let me know if there is anything I can do.

Restarted the CI by hand, I'll look into it if it still fails

I was just trying out with the OAuth token and couldn't exceed the limit in 60 attempts. Not sure why this one failed!

We fixed the CI issue 👍

Fixes #13264 (adding this comment so GitHub creates a link to the issue)

Okay, I checked that the fix works and ran the unit tests: no errors.

Merging.