FrameSelection::updateApperance for caret should not cause layout.

Source/core/frame/FrameView.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Dirk Mueller <mueller@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
@@ skipping 977 matching lines @@
     if (hasVerticalBarDamage())
         invalidateRect(verticalBarDamage());
     if (hasHorizontalBarDamage())
         invalidateRect(horizontalBarDamage());
     resetScrollbarDamage();
 
     m_doFullPaintInvalidation = false;
 #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
 {
     return m_frame->document()->lifecycle();
 }
 
 void FrameView::gatherDebugLayoutRects(RenderObject* layoutRoot)
 {
     bool isTracing;
@@ skipping 2125 matching lines @@
 void FrameView::willRemoveScrollbar(Scrollbar* scrollbar, ScrollbarOrientation orientation)
 {
     ScrollableArea::willRemoveScrollbar(scrollbar, orientation);
     if (AXObjectCache* cache = axObjectCache()) {
         cache->remove(scrollbar);
         cache->handleScrollbarUpdate(this);
     }
 }
 
 } // namespace blink