| Index: Source/core/frame/FrameView.cpp
|
| diff --git a/Source/core/frame/FrameView.cpp b/Source/core/frame/FrameView.cpp
|
| index f145456bb85ac2c64bee43768a27489f3ff878ea..be47b6ca4f54600fd36518b97f27f060f4016a15 100644
|
| --- a/Source/core/frame/FrameView.cpp
|
| +++ b/Source/core/frame/FrameView.cpp
|
| @@ -349,7 +349,7 @@ void FrameView::setFrameRect(const IntRect& newRect)
|
| updateScrollableAreaSet();
|
|
|
| if (autosizerNeedsUpdating) {
|
| - // This needs to be after the call to ScrollView::setFrameRect, because it reads the new width.
|
| + // This needs to be after the call to setFrameRectInternal, because it reads the new width.
|
| if (TextAutosizer* textAutosizer = m_frame->document()->textAutosizer())
|
| textAutosizer->updatePageInfoInAllFrames();
|
| }
|
| @@ -2762,22 +2762,17 @@ IntPoint FrameView::convertToRenderer(const RenderObject& renderer, const IntPoi
|
|
|
| IntRect FrameView::convertToContainingView(const IntRect& localRect) const
|
| {
|
| - if (const FrameView* parentScrollView = toFrameView(parent())) {
|
| - if (parentScrollView->isFrameView()) {
|
| - const FrameView* parentView = parentScrollView;
|
| - // Get our renderer in the parent view
|
| - RenderPart* renderer = m_frame->ownerRenderer();
|
| - if (!renderer)
|
| - return localRect;
|
| -
|
| - IntRect rect(localRect);
|
| - // Add borders and padding??
|
| - rect.move(renderer->borderLeft() + renderer->paddingLeft(),
|
| - renderer->borderTop() + renderer->paddingTop());
|
| - return parentView->convertFromRenderer(*renderer, rect);
|
| - }
|
| -
|
| - return Widget::convertToContainingView(localRect);
|
| + if (const FrameView* parentView = toFrameView(parent())) {
|
| + // Get our renderer in the parent view
|
| + RenderPart* renderer = m_frame->ownerRenderer();
|
| + if (!renderer)
|
| + return localRect;
|
| +
|
| + IntRect rect(localRect);
|
| + // Add borders and padding??
|
| + rect.move(renderer->borderLeft() + renderer->paddingLeft(),
|
| + renderer->borderTop() + renderer->paddingTop());
|
| + return parentView->convertFromRenderer(*renderer, rect);
|
| }
|
|
|
| return localRect;
|
| @@ -2785,23 +2780,17 @@ IntRect FrameView::convertToContainingView(const IntRect& localRect) const
|
|
|
| IntRect FrameView::convertFromContainingView(const IntRect& parentRect) const
|
| {
|
| - if (const FrameView* parentScrollView = toFrameView(parent())) {
|
| - if (parentScrollView->isFrameView()) {
|
| - const FrameView* parentView = parentScrollView;
|
| -
|
| - // Get our renderer in the parent view
|
| - RenderPart* renderer = m_frame->ownerRenderer();
|
| - if (!renderer)
|
| - return parentRect;
|
| -
|
| - IntRect rect = parentView->convertToRenderer(*renderer, parentRect);
|
| - // Subtract borders and padding
|
| - rect.move(-renderer->borderLeft() - renderer->paddingLeft(),
|
| - -renderer->borderTop() - renderer->paddingTop());
|
| - return rect;
|
| - }
|
| -
|
| - return Widget::convertFromContainingView(parentRect);
|
| + if (const FrameView* parentView = toFrameView(parent())) {
|
| + // Get our renderer in the parent view
|
| + RenderPart* renderer = m_frame->ownerRenderer();
|
| + if (!renderer)
|
| + return parentRect;
|
| +
|
| + IntRect rect = parentView->convertToRenderer(*renderer, parentRect);
|
| + // Subtract borders and padding
|
| + rect.move(-renderer->borderLeft() - renderer->paddingLeft(),
|
| + -renderer->borderTop() - renderer->paddingTop());
|
| + return rect;
|
| }
|
|
|
| return parentRect;
|
| @@ -2809,24 +2798,18 @@ IntRect FrameView::convertFromContainingView(const IntRect& parentRect) const
|
|
|
| IntPoint FrameView::convertToContainingView(const IntPoint& localPoint) const
|
| {
|
| - if (const FrameView* parentScrollView = toFrameView(parent())) {
|
| - if (parentScrollView->isFrameView()) {
|
| - const FrameView* parentView = parentScrollView;
|
| -
|
| - // Get our renderer in the parent view
|
| - RenderPart* renderer = m_frame->ownerRenderer();
|
| - if (!renderer)
|
| - return localPoint;
|
| + if (const FrameView* parentView = toFrameView(parent())) {
|
| + // Get our renderer in the parent view
|
| + RenderPart* renderer = m_frame->ownerRenderer();
|
| + if (!renderer)
|
| + return localPoint;
|
|
|
| - IntPoint point(localPoint);
|
| + IntPoint point(localPoint);
|
|
|
| - // Add borders and padding
|
| - point.move(renderer->borderLeft() + renderer->paddingLeft(),
|
| - renderer->borderTop() + renderer->paddingTop());
|
| - return parentView->convertFromRenderer(*renderer, point);
|
| - }
|
| -
|
| - return Widget::convertToContainingView(localPoint);
|
| + // Add borders and padding
|
| + point.move(renderer->borderLeft() + renderer->paddingLeft(),
|
| + renderer->borderTop() + renderer->paddingTop());
|
| + return parentView->convertFromRenderer(*renderer, point);
|
| }
|
|
|
| return localPoint;
|
| @@ -2834,23 +2817,17 @@ IntPoint FrameView::convertToContainingView(const IntPoint& localPoint) const
|
|
|
| IntPoint FrameView::convertFromContainingView(const IntPoint& parentPoint) const
|
| {
|
| - if (const FrameView* parentScrollView = toFrameView(parent())) {
|
| - if (parentScrollView->isFrameView()) {
|
| - const FrameView* parentView = parentScrollView;
|
| -
|
| - // Get our renderer in the parent view
|
| - RenderPart* renderer = m_frame->ownerRenderer();
|
| - if (!renderer)
|
| - return parentPoint;
|
| -
|
| - IntPoint point = parentView->convertToRenderer(*renderer, parentPoint);
|
| - // Subtract borders and padding
|
| - point.move(-renderer->borderLeft() - renderer->paddingLeft(),
|
| - -renderer->borderTop() - renderer->paddingTop());
|
| - return point;
|
| - }
|
| -
|
| - return Widget::convertFromContainingView(parentPoint);
|
| + if (const FrameView* parentView = toFrameView(parent())) {
|
| + // Get our renderer in the parent view
|
| + RenderPart* renderer = m_frame->ownerRenderer();
|
| + if (!renderer)
|
| + return parentPoint;
|
| +
|
| + IntPoint point = parentView->convertToRenderer(*renderer, parentPoint);
|
| + // Subtract borders and padding
|
| + point.move(-renderer->borderLeft() - renderer->paddingLeft(),
|
| + -renderer->borderTop() - renderer->paddingTop());
|
| + return point;
|
| }
|
|
|
| return parentPoint;
|
| @@ -3055,8 +3032,6 @@ IntPoint FrameView::maximumScrollPosition() const
|
| return maximumOffset;
|
| }
|
|
|
| -// --- ScrollView ---
|
| -
|
| void FrameView::addChild(PassRefPtr<Widget> prpChild)
|
| {
|
| Widget* child = prpChild.get();
|
|
|