Patch to follow
Created attachment 194484 [details] Patch
Comment on attachment 194484 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=194484&action=review > Source/WebCore/inspector/front-end/CodeMirrorTextEditor.js:78 > + this._codeMirror.on("beforeSelectionChange", this._selectionChange.bind(this)); There's already a selection listener in ToT, but it observes "cursorActivity" event. > Source/WebCore/inspector/front-end/CodeMirrorTextEditor.js:324 > + _coordsChar: function(coords) Maybe "coordsCharLocal"? > Source/WebCore/inspector/front-end/CodeMirrorTextEditor.js:332 > + var middle = (begin + end) / 2; > + middle = middle >>> 0; maybe (begin + end) >> 1 ?
Committed r146615: <http://trac.webkit.org/changeset/146615>