RESOLVED FIXED 138791
Avoid synchronous style recalc when mutating a Node inside FrameSelection. 
https://bugs.webkit.org/show_bug.cgi?id=138791
Summary Avoid synchronous style recalc when mutating a Node inside FrameSelection.
Andreas Kling
Reported 2014-11-16 23:59:03 PST
There's no need to force a synchronous style recalc before calling RenderView::clearSelection(). FrameSelection is very trigger happy with this.
Attachments
Patch idea (7.46 KB, patch)
2014-11-17 00:28 PST, Andreas Kling 		koivisto: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2014-11-17 00:28:05 PST
Created attachment 241696 [details] Patch idea
Antti Koivisto
Comment 2 2014-11-17 02:14:45 PST
Comment on attachment 241696 [details] Patch idea View in context: https://bugs.webkit.org/attachment.cgi?id=241696&action=review > Source/WebCore/editing/FrameSelection.cpp:-485 > + if (RenderView* renderView = node.document().renderView()) { > + renderView->clearSelection(); > > - // Trigger a selection update so the selection will be set again. > - if (auto* renderView = node.document().renderView()) { why auto* -> RenderView*?
Andreas Kling
Comment 3 2014-11-17 08:22:33 PST
Committed r176201: <http://trac.webkit.org/changeset/176201>
Note You need to log in before you can comment on or make changes to this bug.