Delete selection paint invalidation code.

sky/engine/core/editing/FrameSelection.cpp

 /*
  * Copyright (C) 2004, 2008, 2009, 2010 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 1502 matching lines @@
         m_frame->document()->setFocusedElement(nullptr);
     }
 }
 
 String FrameSelection::selectedText() const
 {
     // We remove '\0' characters because they are not visibly rendered to the user.
     return plainText(toNormalizedRange().get()).replace(0, "");
 }
 
-FloatRect FrameSelection::bounds() const
-{
-    m_frame->document()->updateRenderTreeIfNeeded();
-
-    FrameView* view = m_frame->view();
-    RenderView* renderView = m_frame->contentRenderer();
-
-    if (!view || !renderView)
-        return FloatRect();
-
-    return renderView->selectionBounds();
-}
-
 void FrameSelection::revealSelection(const ScrollAlignment& alignment, RevealExtentOption revealExtentOption)
 {
     LayoutRect rect;
 
     switch (selectionType()) {
     case NoSelection:
         return;
     case CaretSelection:
         rect = absoluteCaretBounds();
         break;
     case RangeSelection:
-        rect = revealExtentOption == RevealExtent ? VisiblePosition(extent()).absoluteCaretBounds() : enclosingIntRect(bounds());
+        Position position = revealExtentOption == RevealExtent ? extent() : base();

[ojan, 2015/01/14 19:49:19]

This is the only change in behavior. The rest of this patch is all deleting dead
code.

+        rect = VisiblePosition(position).absoluteCaretBounds();
         break;
     }
 
     Position start = this->start();
     ASSERT(start.deprecatedNode());
     if (start.deprecatedNode() && start.deprecatedNode()->renderer()) {
         // FIXME: This code only handles scrolling the startContainer's layer, but
         // the selection rect could intersect more than just that.
         // See <rdar://problem/4799899>.
         if (start.deprecatedNode()->renderer()->scrollRectToVisible(rect, alignment, alignment))
@@ skipping 109 matching lines @@
     sel.showTreeForThis();
 }
 
 void showTree(const blink::FrameSelection* sel)
 {
     if (sel)
         sel->showTreeForThis();
 }
 
 #endif