Are we sure that that is a regression ? Because if a take a peek at textSelection, it seems like it would have done that in the past as well..

Will have to investigate a bit.

(In reply to Derk-Jan Hartman from comment #1)

Are we sure that that is a regression ? Because if a take a peek at
textSelection, it seems like it would have done that in the past as well..

Yes, I am sure. I have CodeEditor disabled, and use several user scripts that use the textSelection API, and they just broke yesterday.

After a quick look: When you disable CodeEditor, context.$iframe is set to undefined [1], which previously was recognized by textSeclection as to use the default methods, while the mentioned change removed that check. So now the code always tries to call the methods provided by CodeEditor, but these fail when it is disabled.

[1] https://git.wikimedia.org/blob/mediawiki%2Fextensions%2FCodeEditor/56b9e4c350c122e105949b0faefa25f17253335b/modules%2Fjquery.codeEditor.js#L276

Bingo got it. Now to find a better way to fix this....
and more importantly.. document it...

Was https://gerrit.wikimedia.org/r/#/c/149529/ a fix for this bug?

Yuck...

This is really all down to the fact that the textSelection api was once a part of the WikiEditor api.

That separation was never really properly finished and that is why in the CodeEditor this api is now 'mixed' up with the WikiEditor api. Add to this that CodeEditor made it possible to enable and disable on the fly and you have one big mess...

So this bug is because I failed to realize that the CodeEditor could be disabled in the same session and that this would 'break' the function mapping in:

http://git.wikimedia.org/blob/mediawiki%2Fextensions%2FCodeEditor.git/ad908b2da64da4083e10e391153112978fb9df8f/modules%2Fjquery.codeEditor.js#L517

There we basically assign the 'textSelection' api functions into the WikiEditor api. The disabling works fine for the WikiEditor API functions, because they listen to the conditional in L508, but the textSelection APIs are no longer part of the wikiEditor api's and stuff start breaking.

I'm thus going to reintroduce iframe check and then submit a new patch that does some very simple API registration to replace the textSelection API if a textfield/area wants to do that.

Change 150009 had a related patch set uploaded by TheDJ:
jquery.textSelection: re-add iframe check due to regression

https://gerrit.wikimedia.org/r/150009

Change 150009 merged by jenkins-bot:
jquery.textSelection: re-add iframe check due to regression

https://gerrit.wikimedia.org/r/150009