FrameSelection::updateApperance for caret should not cause layout.

Source/web/WebViewImpl.cpp

Index: Source/web/WebViewImpl.cpp
diff --git a/Source/web/WebViewImpl.cpp b/Source/web/WebViewImpl.cpp
index 9f8c31ff244cd5c6fcd84ac10f547d188cc6ce3f..bc5395c52e9a3d835a571fc65cfa8068d50e92c6 100644
--- a/Source/web/WebViewImpl.cpp
+++ b/Source/web/WebViewImpl.cpp
@@ -1742,6 +1742,11 @@ void WebViewImpl::beginFrame(const WebBeginFrameArgs& frameTime)
         ContinuousPainter::setNeedsDisplayRecursive(m_rootGraphicsLayer, m_pageOverlays.get());
         m_client->scheduleAnimation();
     }
+
+    const Frame* frame = focusedWebCoreFrame();
+    if (frame && frame->isLocalFrame()) {

[yosin_UTC9, 2014/08/07 01:09:55]

nit: We don't need to have "{}" for single line then clause.

[yoichio, 2014/08/07 03:03:06]

Done.

+        toLocalFrame(frame)->selection().recomputeCaretRect();
+    }
 }
 
 void WebViewImpl::layout()