Rename Document::widgetForElement to frameViewBaseForElement

third_party/WebKit/Source/core/dom/Node.cpp

Index: third_party/WebKit/Source/core/dom/Node.cpp
diff --git a/third_party/WebKit/Source/core/dom/Node.cpp b/third_party/WebKit/Source/core/dom/Node.cpp
index b8c3eb497fd360ab5fcc1e67c7bb0509642f0390..add82d65035434356372c09455066ab444d04c94 100644
--- a/third_party/WebKit/Source/core/dom/Node.cpp
+++ b/third_party/WebKit/Source/core/dom/Node.cpp
@@ -92,6 +92,7 @@
 #include "core/html/HTMLSlotElement.h"
 #include "core/input/EventHandler.h"
 #include "core/layout/LayoutBox.h"
+#include "core/layout/LayoutPart.h"
 #include "core/page/ContextMenuController.h"
 #include "core/page/Page.h"
 #include "core/svg/SVGElement.h"
@@ -594,6 +595,13 @@ LayoutBoxModelObject* Node::layoutBoxModelObject() const {
              : nullptr;
 }
 
+FrameViewBase* Node::frameViewBase() const {
+  LayoutObject* layoutObject = this->layoutObject();
+  return layoutObject && layoutObject->isLayoutPart()

[sashab, 2017/03/10 04:30:13]

I actually don't know if this is easier to read :( The early exit was simple and
matches the rest of the codebase... Can we leave it that way? :)

[joelhockey, 2017/03/10 04:37:53]

I wrote it to match the surrounding, related code.  See the 2 methods above it.

[sashab, 2017/03/10 04:47:22]

Oh, so you did, sorry about that :)

+             ? toLayoutPart(layoutObject)->frameViewBase()
+             : nullptr;
+}
+
 LayoutRect Node::boundingBox() const {
   if (layoutObject())
     return LayoutRect(layoutObject()->absoluteBoundingBoxRect());