Follow-up work for 151157 to address other uses of kill ring.
Created attachment 266030 [details] Proposed Fix
<rdar://problem/23642146>
<rdar://problem/23642148>
Comment on attachment 266030 [details] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=266030&action=review > Source/WebInspectorUI/UserInterface/Controllers/CodeMirrorTextKillController.js:44 > + // FIXME: these keybindings match CodeMirror's default keymap for OS X. > + // They should probably be altered for Windows / Linux someday. Unfortunate to have to hard code these like this. Generally speaking, Safari and WebKit respect the OS X system-wide key bindings <https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/EventOverview/TextDefaultsBindings/TextDefaultsBindings.html>.
Comment on attachment 266030 [details] Proposed Fix Clearing flags on attachment: 266030 Committed r192804: <http://trac.webkit.org/changeset/192804>
All reviewed patches have been landed. Closing bug.
(In reply to comment #4) > Comment on attachment 266030 [details] > Proposed Fix > > View in context: > https://bugs.webkit.org/attachment.cgi?id=266030&action=review > > > Source/WebInspectorUI/UserInterface/Controllers/CodeMirrorTextKillController.js:44 > > + // FIXME: these keybindings match CodeMirror's default keymap for OS X. > > + // They should probably be altered for Windows / Linux someday. > > Unfortunate to have to hard code these like this. Generally speaking, Safari > and WebKit respect the OS X system-wide key bindings > <https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/ > EventOverview/TextDefaultsBindings/TextDefaultsBindings.html>. Wow, I had no idea that this exists! I'll file a bug to see if we can integrate it.