When Firepad is disposed, CodeMirror undo/redo don't return to their original state

[mig8447]

Firepad replaces CodeMirror redo/undo functions with its own handlers for undo and redo but when disposing Firepad, it doesn't return them to their original functions and this causes unexpected behavior in CodeMirror, leaving it in an inconsistent state leading to potential data loss

See https://github.com/FirebaseExtended/firepad/blob/master/lib/rich-text-codemirror-adapter.js#L299 and https://github.com/FirebaseExtended/firepad/blob/master/lib/rich-text-codemirror-adapter.js#L303

When detaching the CodeMirror adapter, those functions should return to their original state. See:

firepad/lib/rich-text-codemirror-adapter.js

Line 29 in 602138d

RichTextCodeMirrorAdapter.prototype.detach = function () {