Remove use of view() in RenderView code
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 10 Jun 2014 05:49:57 +0000 (05:49 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 10 Jun 2014 05:49:57 +0000 (05:49 +0000)
https://bugs.webkit.org/show_bug.cgi?id=133669

Reviewed by Andreas Kling.

view() in RenderView just returns *this, so no need to use it.

* rendering/RenderView.cpp:
(WebCore::RenderView::setSelection):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@169742 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderView.cpp

index d923b74..f7e27f4 100644 (file)
@@ -1,3 +1,15 @@
+2014-06-09  Simon Fraser  <simon.fraser@apple.com>
+
+        Remove use of view() in RenderView code
+        https://bugs.webkit.org/show_bug.cgi?id=133669
+
+        Reviewed by Andreas Kling.
+
+        view() in RenderView just returns *this, so no need to use it.
+
+        * rendering/RenderView.cpp:
+        (WebCore::RenderView::setSelection):
+
 2014-06-09  Eric Carlson  <eric.carlson@apple.com>
 
         [iOS] Wireless route picker control should use touch event handlers
index 834aba6..26d52d0 100644 (file)
@@ -839,9 +839,8 @@ void RenderView::setSelection(RenderObject* start, int startPos, RenderObject* e
     if ((start && !end) || (end && !start))
         return;
 
-    // FIXME: view() is just *this here.
-    bool caretChanged = m_selectionWasCaret != view().frame().selection().isCaret();
-    m_selectionWasCaret = view().frame().selection().isCaret();
+    bool caretChanged = m_selectionWasCaret != frame().selection().isCaret();
+    m_selectionWasCaret = frame().selection().isCaret();
     // Just return if the selection hasn't changed.
     if (m_selectionStart == start && m_selectionStartPos == startPos &&
         m_selectionEnd == end && m_selectionEndPos == endPos && !caretChanged)