Remove unneeded Convert* methods and move many from FrameViewBase to FrameView

third_party/WebKit/Source/core/frame/FrameView.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Dirk Mueller <mueller@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
@@ skipping 3605 matching lines @@
 IntPoint FrameView::ConvertToLayoutItem(const LayoutItem& layout_item,
                                         const IntPoint& frame_point) const {
   IntPoint point = frame_point;
 
   // Convert from FrameView coords into page ("absolute") coordinates.
   point += IntSize(ScrollX(), ScrollY());
 
   return RoundedIntPoint(layout_item.AbsoluteToLocal(point, kUseTransforms));
 }
 
+IntPoint FrameView::ConvertSelfToChild(const FrameViewBase* child,
+                                       const IntPoint& point) const {
+  // TODO(joelhockey): Remove this check once Scrollbar no longer inherits from
+  // FrameViewBase.
+  DCHECK(!IsFrameViewScrollbar(child));
+  IntPoint new_point = point;
+  new_point = FrameToContents(point);
+  new_point.MoveBy(-child->Location());
+  return new_point;
+}
+
 IntRect FrameView::ConvertToContainingFrameViewBase(
     const IntRect& local_rect) const {
-  if (const FrameView* parent_view = ToFrameView(Parent())) {
+  if (const FrameView* parent = ToFrameView(Parent())) {
     // Get our layoutObject in the parent view
     LayoutPartItem layout_item = frame_->OwnerLayoutItem();
     if (layout_item.IsNull())
       return local_rect;
 
     IntRect rect(local_rect);
     // Add borders and padding??
     rect.Move((layout_item.BorderLeft() + layout_item.PaddingLeft()).ToInt(),
               (layout_item.BorderTop() + layout_item.PaddingTop()).ToInt());
-    return parent_view->ConvertFromLayoutItem(layout_item, rect);
+    return parent->ConvertFromLayoutItem(layout_item, rect);
   }
 
   return local_rect;
 }
 
 IntRect FrameView::ConvertFromContainingFrameViewBase(
     const IntRect& parent_rect) const {
-  if (const FrameView* parent_view = ToFrameView(Parent())) {
-    // Get our layoutObject in the parent view
-    LayoutPartItem layout_item = frame_->OwnerLayoutItem();
-    if (layout_item.IsNull())
-      return parent_rect;
-
-    IntRect rect = parent_view->ConvertToLayoutItem(layout_item, parent_rect);
-    // Subtract borders and padding
-    rect.Move((-layout_item.BorderLeft() - layout_item.PaddingLeft()).ToInt(),
-              (-layout_item.BorderTop() - layout_item.PaddingTop()).ToInt());
-    return rect;
+  if (const FrameView* parent = ToFrameView(Parent())) {
+    IntRect local_rect = parent_rect;
+    local_rect.SetLocation(
+        parent->ConvertSelfToChild(this, local_rect.Location()));
+    return local_rect;
   }
 
   return parent_rect;
 }
 
 IntPoint FrameView::ConvertToContainingFrameViewBase(
     const IntPoint& local_point) const {
-  if (const FrameView* parent_view = ToFrameView(Parent())) {
+  if (const FrameView* parent = ToFrameView(Parent())) {
     // Get our layoutObject in the parent view
     LayoutPartItem layout_item = frame_->OwnerLayoutItem();
     if (layout_item.IsNull())
       return local_point;
 
     IntPoint point(local_point);
 
     // Add borders and padding
     point.Move((layout_item.BorderLeft() + layout_item.PaddingLeft()).ToInt(),
                (layout_item.BorderTop() + layout_item.PaddingTop()).ToInt());
-    return parent_view->ConvertFromLayoutItem(layout_item, point);
+    return parent->ConvertFromLayoutItem(layout_item, point);
   }
 
   return local_point;
 }
 
 IntPoint FrameView::ConvertFromContainingFrameViewBase(
     const IntPoint& parent_point) const {
-  if (const FrameView* parent_view = ToFrameView(Parent())) {
+  if (const FrameView* parent = ToFrameView(Parent())) {
     // Get our layoutObject in the parent view
     LayoutPartItem layout_item = frame_->OwnerLayoutItem();
     if (layout_item.IsNull())
       return parent_point;
 
-    IntPoint point =
-        parent_view->ConvertToLayoutItem(layout_item, parent_point);
+    IntPoint point = parent->ConvertToLayoutItem(layout_item, parent_point);
     // Subtract borders and padding
     point.Move((-layout_item.BorderLeft() - layout_item.PaddingLeft()).ToInt(),
                (-layout_item.BorderTop() - layout_item.PaddingTop()).ToInt());
     return point;
   }
 
   return parent_point;
 }
 
 void FrameView::SetInitialTracksPaintInvalidationsForTesting(
@@ skipping 109 matching lines @@
     animating_scrollable_areas_ = new ScrollableAreaSet;
   animating_scrollable_areas_->insert(scrollable_area);
 }
 
 void FrameView::RemoveAnimatingScrollableArea(ScrollableArea* scrollable_area) {
   if (!animating_scrollable_areas_)
     return;
   animating_scrollable_areas_->erase(scrollable_area);
 }
 
-void FrameView::SetParent(FrameViewBase* parent_view) {
-  if (parent_view == Parent())
+void FrameView::SetParent(FrameViewBase* parent) {
+  if (parent == Parent())
     return;
 
-  FrameViewBase::SetParent(parent_view);
+  FrameViewBase::SetParent(parent);
 
   UpdateParentScrollableAreaSet();
   SetupRenderThrottling();
 
   if (ParentFrameView())
     subtree_throttled_ = ParentFrameView()->CanThrottleRendering();
 }
 
 void FrameView::RemoveChild(FrameViewBase* child) {
   DCHECK(child->Parent() == this);
@@ skipping 856 matching lines @@
          frame_point.X() < vertical_scrollbar_x_max &&
          frame_point.Y() > vertical_scrollbar_y_min;
 }
 
 bool FrameView::ScrollbarCornerPresent() const {
   return (HorizontalScrollbar() &&
           Width() - HorizontalScrollbar()->Width() > 0) ||
          (VerticalScrollbar() && Height() - VerticalScrollbar()->Height() > 0);
 }
 
-IntRect FrameView::ConvertFromScrollbarToContainingFrameViewBase(
-    const Scrollbar& scrollbar,
-    const IntRect& local_rect) const {
-  // Scrollbars won't be transformed within us
-  IntRect new_rect = local_rect;
-  new_rect.MoveBy(scrollbar.Location());
-  return new_rect;
+IntRect FrameView::ConvertToRootFrame(const IntRect& local_rect) const {
+  if (const FrameView* parent = ToFrameView(Parent())) {
+    IntRect parent_rect = ConvertToContainingFrameViewBase(local_rect);
+    return parent->ConvertToRootFrame(parent_rect);
+  }
+  return local_rect;
 }
 
-IntRect FrameView::ConvertFromContainingFrameViewBaseToScrollbar(
-    const Scrollbar& scrollbar,
-    const IntRect& parent_rect) const {
-  IntRect new_rect = parent_rect;
-  // Scrollbars won't be transformed within us
-  new_rect.MoveBy(-scrollbar.Location());
-  return new_rect;
-}
-
-// FIXME: test these on windows
-IntPoint FrameView::ConvertFromScrollbarToContainingFrameViewBase(
-    const Scrollbar& scrollbar,
-    const IntPoint& local_point) const {
-  // Scrollbars won't be transformed within us
-  IntPoint new_point = local_point;
-  new_point.MoveBy(scrollbar.Location());
-  return new_point;
+IntPoint FrameView::ConvertToRootFrame(const IntPoint& local_point) const {
+  if (const FrameView* parent = ToFrameView(Parent())) {
+    IntPoint parent_point = ConvertToContainingFrameViewBase(local_point);
+    return parent->ConvertToRootFrame(parent_point);
+  }
+  return local_point;
 }
 
 IntPoint FrameView::ConvertFromContainingFrameViewBaseToScrollbar(
     const Scrollbar& scrollbar,
     const IntPoint& parent_point) const {
   IntPoint new_point = parent_point;
   // Scrollbars won't be transformed within us
   new_point.MoveBy(-scrollbar.Location());
   return new_point;
 }
@@ skipping 613 matching lines @@
 void FrameView::SetAnimationHost(
     std::unique_ptr<CompositorAnimationHost> host) {
   animation_host_ = std::move(host);
 }
 
 LayoutUnit FrameView::CaretWidth() const {
   return LayoutUnit(GetChromeClient()->WindowToViewportScalar(1));
 }
 
 }  // namespace blink