LayoutObject::absoluteBoundingBoxRectForRange() should take EphemeralRange

third_party/WebKit/Source/web/FindInPageCoordinates.cpp

 /*
  * Copyright (C) 2012 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 125 matching lines @@
     DCHECK(layoutObject->isLayoutView());
 
     // Jump to the layoutObject owning the frame, if any.
     layoutObject =
         layoutObject->frame() ? layoutObject->frame()->ownerLayoutObject() : 0;
   }
 
   return normalizedRect;
 }
 
-FloatRect findInPageRectFromRange(Range* range) {
-  if (!range || !range->firstNode())
+FloatRect findInPageRectFromRange(EphemeralRange& range) {
+  if (range.isNull() || !range.startPosition().nodeAsRangeFirstNode())

[Xiaocheng, 2017/03/24 18:19:26]

Could you store the result of |range.startPosition().nodeAsRangeFirstNode()| to
eliminate repeated computation?

     return FloatRect();
 
   return findInPageRectFromAbsoluteRect(
       LayoutObject::absoluteBoundingBoxRectForRange(range),
-      range->firstNode()->layoutObject());
+      range.startPosition().nodeAsRangeFirstNode()->layoutObject());
 }
 
 }  // namespace blink