Make Position::LastOffsetInNode() to take const Node&

third_party/WebKit/Source/core/editing/Position.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2009 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 170 matching lines @@
 
 template <typename Strategy>
 int PositionTemplate<Strategy>::ComputeOffsetInContainerNode() const {
   if (!anchor_node_)
     return 0;
 
   switch (AnchorType()) {
     case PositionAnchorType::kBeforeChildren:
       return 0;
     case PositionAnchorType::kAfterChildren:
-      return LastOffsetInNode(anchor_node_.Get());
+      return LastOffsetInNode(*anchor_node_);
     case PositionAnchorType::kOffsetInAnchor:
       return MinOffsetForNode<Strategy>(anchor_node_.Get(), offset_);
     case PositionAnchorType::kBeforeAnchor:
       return Strategy::Index(*anchor_node_);
     case PositionAnchorType::kAfterAnchor:
       return Strategy::Index(*anchor_node_) + 1;
   }
   NOTREACHED();
   return 0;
 }
@@ skipping 272 matching lines @@
 // static
 template <typename Strategy>
 PositionTemplate<Strategy> PositionTemplate<Strategy>::AfterNode(
     const Node& anchor_node) {
   return PositionTemplate<Strategy>(&anchor_node,
                                     PositionAnchorType::kAfterAnchor);
 }
 
 // static
 template <typename Strategy>
-int PositionTemplate<Strategy>::LastOffsetInNode(Node* node) {
-  return node->IsCharacterDataNode()
-             ? node->MaxCharacterOffset()
-             : static_cast<int>(Strategy::CountChildren(*node));
+int PositionTemplate<Strategy>::LastOffsetInNode(const Node& node) {
+  return node.IsCharacterDataNode()
+             ? node.MaxCharacterOffset()
+             : static_cast<int>(Strategy::CountChildren(node));
 }
 
 // static
 template <typename Strategy>
 PositionTemplate<Strategy> PositionTemplate<Strategy>::FirstPositionInNode(
     const Node& anchor_node) {
   if (anchor_node.IsTextNode())
     return PositionTemplate<Strategy>(anchor_node, 0);
   return PositionTemplate<Strategy>(&anchor_node,
                                     PositionAnchorType::kBeforeChildren);
 }
 
 // static
 template <typename Strategy>
 PositionTemplate<Strategy> PositionTemplate<Strategy>::LastPositionInNode(
     Node* anchor_node) {
-  if (anchor_node->IsTextNode())
+  if (anchor_node->IsTextNode()) {
     return PositionTemplate<Strategy>(anchor_node,
-                                      LastOffsetInNode(anchor_node));
+                                      LastOffsetInNode(*anchor_node));
+  }
   return PositionTemplate<Strategy>(anchor_node,
                                     PositionAnchorType::kAfterChildren);
 }
 
 // static
 template <typename Strategy>
 PositionTemplate<Strategy>
 PositionTemplate<Strategy>::FirstPositionInOrBeforeNode(Node* node) {
   if (!node)
     return PositionTemplate<Strategy>();
@@ skipping 197 matching lines @@
 }
 
 void showTree(const blink::Position* pos) {
   if (pos)
     pos->ShowTreeForThis();
   else
     LOG(INFO) << "Cannot showTree for <null>";
 }
 
 #endif