RESOLVED FIXED 82733
Web Inspector: undo-ing edit that consists of a Tab does not work.
https://bugs.webkit.org/show_bug.cgi?id=82733
Summary Web Inspector: undo-ing edit that consists of a Tab does not work.
Pavel Feldman
Reported 2012-03-30 08:39:36 PDT
We should never modify the range returned by the edit operation manually. And we should clone ranges that get into the model so that subsequent edits don't mutate them.
Attachments
Patch (5.22 KB, patch)
2012-03-30 08:48 PDT, Pavel Feldman
no flags
Patch (5.36 KB, patch)
2012-03-30 09:15 PDT, Pavel Feldman
no flags
[Patch] with selection restored (5.76 KB, patch)
2012-03-30 09:21 PDT, Pavel Feldman
vsevik: review+
Pavel Feldman
Comment 1 2012-03-30 08:48:43 PDT
Vsevolod Vlasov
Comment 2 2012-03-30 09:11:44 PDT
Comment on attachment 134820 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=134820&action=review > Source/WebCore/inspector/front-end/TextViewer.js:1134 > + newRange.startColumn = newRange.endColumn; I think this is not supposed to happen in some of the code flows above.
Pavel Feldman
Comment 3 2012-03-30 09:15:24 PDT
Pavel Feldman
Comment 4 2012-03-30 09:21:42 PDT
Created attachment 134826 [details] [Patch] with selection restored
Pavel Feldman
Comment 5 2012-03-30 09:24:56 PDT
Note You need to log in before you can comment on or make changes to this bug.