FrameSelection::updateApperance for caret should not cause layout.

Source/core/frame/FrameView.cpp

Index: Source/core/frame/FrameView.cpp
diff --git a/Source/core/frame/FrameView.cpp b/Source/core/frame/FrameView.cpp
index 055dc003990c16754a31d390ba5aa50fb153c637..3c8e9762425af206ab1ec4d2dc9ca42f89540986 100644
--- a/Source/core/frame/FrameView.cpp
+++ b/Source/core/frame/FrameView.cpp
@@ -995,6 +995,12 @@ void FrameView::invalidateTreeIfNeeded()
 #ifndef NDEBUG
     renderView()->assertSubtreeClearedPaintInvalidationState();
 #endif
+
+    Page* page = frame().page();
+    ASSERT(page);
+    LocalFrame* frame = toLocalFrame(page->focusController().focusedOrMainFrame());
+    if (frame)
+        frame->selection().invalidateCaretRect();

[abarth-chromium, 2014/08/12 19:36:25]

Suppose the caret moved from one LocalFrame to another during this animation
frame.  Where do we invalidate the old caret location?  You didn't use the
design I suggested.

[yoichio, 2014/08/13 20:05:28]

Call each LocalFrame invalidation.
Thus a focused out LocalFrame invalidates old caret location 
and a focued in LocalFrame invalidates new caret location.

 }
 
 DocumentLifecycle& FrameView::lifecycle() const