Refactoring code in highestEditableRoot to avoid a redundant call to rendererIsEditable

Source/core/editing/htmlediting.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007 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 98 matching lines @@
 Node* highestEditableRoot(const Position& position, EditableType editableType)
 {
     Node* node = position.deprecatedNode();
     if (!node)
         return 0;
 
     Node* highestRoot = editableRootForPosition(position, editableType);
     if (!highestRoot)
         return 0;
 
-    node = highestRoot;
+    if (highestRoot->hasTagName(bodyTag))
+        return highestRoot;
+
+    node = highestRoot->parentNode();
     while (node) {
         if (node->rendererIsEditable(editableType))
             highestRoot = node;
         if (node->hasTagName(bodyTag))
             break;
         node = node->parentNode();
     }
 
     return highestRoot;
 }
@@ skipping 1030 matching lines @@
     // if the selection starts just before a paragraph break, skip over it
     if (isEndOfParagraph(visiblePosition))
         return visiblePosition.next().deepEquivalent().downstream();
 
     // otherwise, make sure to be at the start of the first selected node,
     // instead of possibly at the end of the last node before the selection
     return visiblePosition.deepEquivalent().downstream();
 }
 
 } // namespace WebCore