| Index: third_party/WebKit/Source/core/frame/FrameView.cpp
|
| diff --git a/third_party/WebKit/Source/core/frame/FrameView.cpp b/third_party/WebKit/Source/core/frame/FrameView.cpp
|
| index de923eecb350bf4c8fe65778e6a205192c500dd0..14e57eeccfd40fc043331754f016791f5cf61bd2 100644
|
| --- a/third_party/WebKit/Source/core/frame/FrameView.cpp
|
| +++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
|
| @@ -159,16 +159,16 @@ FrameView::FrameView(LocalFrame* frame)
|
| init();
|
| }
|
|
|
| -PassRefPtrWillBeRawPtr<FrameView> FrameView::create(LocalFrame* frame)
|
| +RawPtr<FrameView> FrameView::create(LocalFrame* frame)
|
| {
|
| - RefPtrWillBeRawPtr<FrameView> view = adoptRefWillBeNoop(new FrameView(frame));
|
| + RawPtr<FrameView> view = (new FrameView(frame));
|
| view->show();
|
| return view.release();
|
| }
|
|
|
| -PassRefPtrWillBeRawPtr<FrameView> FrameView::create(LocalFrame* frame, const IntSize& initialSize)
|
| +RawPtr<FrameView> FrameView::create(LocalFrame* frame, const IntSize& initialSize)
|
| {
|
| - RefPtrWillBeRawPtr<FrameView> view = adoptRefWillBeNoop(new FrameView(frame));
|
| + RawPtr<FrameView> view = (new FrameView(frame));
|
| view->Widget::setFrameRect(IntRect(view->location(), initialSize));
|
| view->setLayoutSizeInternal(initialSize);
|
|
|
| @@ -348,7 +348,7 @@ void FrameView::invalidateAllCustomScrollbarsOnActiveChanged()
|
| bool usesWindowInactiveSelector = m_frame->document()->styleEngine().usesWindowInactiveSelector();
|
|
|
| const ChildrenWidgetSet* viewChildren = children();
|
| - for (const RefPtrWillBeMember<Widget>& child : *viewChildren) {
|
| + for (const Member<Widget>& child : *viewChildren) {
|
| Widget* widget = child.get();
|
| if (widget->isFrameView())
|
| toFrameView(widget)->invalidateAllCustomScrollbarsOnActiveChanged();
|
| @@ -509,7 +509,7 @@ bool FrameView::shouldUseCustomScrollbars(Element*& customScrollbarElement, Loca
|
| return false;
|
| }
|
|
|
| -PassRefPtrWillBeRawPtr<Scrollbar> FrameView::createScrollbar(ScrollbarOrientation orientation)
|
| +RawPtr<Scrollbar> FrameView::createScrollbar(ScrollbarOrientation orientation)
|
| {
|
| Element* customScrollbarElement = nullptr;
|
| LocalFrame* customScrollbarFrame = nullptr;
|
| @@ -754,7 +754,7 @@ inline void FrameView::forceLayoutParentViewIfNeeded()
|
| // FrameView for a layout. After that the LayoutEmbeddedObject (ownerLayoutObject) carries the
|
| // correct size, which LayoutSVGRoot::computeReplacedLogicalWidth/Height rely on, when laying
|
| // out for the first time, or when the LayoutSVGRoot size has changed dynamically (eg. via <script>).
|
| - RefPtrWillBeRawPtr<FrameView> frameView = ownerLayoutObject->frame()->view();
|
| + RawPtr<FrameView> frameView = ownerLayoutObject->frame()->view();
|
|
|
| // Mark the owner layoutObject as needing layout.
|
| ownerLayoutObject->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation(LayoutInvalidationReason::Unknown);
|
| @@ -924,7 +924,7 @@ void FrameView::layout()
|
| TRACE_EVENT_SCOPED_SAMPLING_STATE("blink", "Layout");
|
|
|
| // Protect the view from being deleted during layout (in recalcStyle)
|
| - RefPtrWillBeRawPtr<FrameView> protector(this);
|
| + RawPtr<FrameView> protector(this);
|
|
|
| if (m_autoSizeInfo)
|
| m_autoSizeInfo->autoSizeIfNeeded();
|
| @@ -1620,7 +1620,7 @@ void FrameView::updateLayersAndCompositingAfterScrollIfNeeded()
|
| if (!hasViewportConstrainedObjects())
|
| return;
|
|
|
| - RefPtrWillBeRawPtr<FrameView> protect(this);
|
| + RawPtr<FrameView> protect(this);
|
|
|
| // If there fixed position elements, scrolling may cause compositing layers to change.
|
| // Update widget and layer positions after scrolling, but only if we're not inside of
|
| @@ -1915,7 +1915,7 @@ void FrameView::updateBackgroundRecursively(const Color& backgroundColor, bool t
|
|
|
| void FrameView::scrollToFragmentAnchor()
|
| {
|
| - RefPtrWillBeRawPtr<Node> anchorNode = m_fragmentAnchor;
|
| + RawPtr<Node> anchorNode = m_fragmentAnchor;
|
| if (!anchorNode)
|
| return;
|
|
|
| @@ -1933,7 +1933,7 @@ void FrameView::scrollToFragmentAnchor()
|
| rect = documentElement->boundingBox();
|
| }
|
|
|
| - RefPtrWillBeRawPtr<Frame> boundaryFrame = m_frame->findUnsafeParentScrollPropagationBoundary();
|
| + RawPtr<Frame> boundaryFrame = m_frame->findUnsafeParentScrollPropagationBoundary();
|
|
|
| // FIXME: Handle RemoteFrames
|
| if (boundaryFrame && boundaryFrame->isLocalFrame())
|
| @@ -1997,7 +1997,7 @@ bool FrameView::updateWidgets()
|
| void FrameView::updateWidgetsTimerFired(Timer<FrameView>*)
|
| {
|
| ASSERT(!isInPerformLayout());
|
| - RefPtrWillBeRawPtr<FrameView> protect(this);
|
| + RawPtr<FrameView> protect(this);
|
| m_updateWidgetsTimer.stop();
|
| for (unsigned i = 0; i < maxUpdateWidgetsIterations; ++i) {
|
| if (updateWidgets())
|
| @@ -2030,7 +2030,7 @@ void FrameView::performPostLayoutTasks()
|
| // We should ASSERT(isActive()); or at least return early if we can!
|
| ASSERT(!isInPerformLayout()); // Always before or after performLayout(), part of the highest-level layout() call.
|
| TRACE_EVENT0("blink,benchmark", "FrameView::performPostLayoutTasks");
|
| - RefPtrWillBeRawPtr<FrameView> protect(this);
|
| + RawPtr<FrameView> protect(this);
|
|
|
| m_postLayoutTasksTimer.stop();
|
|
|
| @@ -2445,7 +2445,7 @@ void FrameView::updateLifecyclePhasesInternal(LifeCycleUpdateOption phases)
|
| ASSERT(m_frame->isLocalRoot());
|
|
|
| // Updating layout can run script, which can tear down the FrameView.
|
| - RefPtrWillBeRawPtr<FrameView> protector(this);
|
| + RawPtr<FrameView> protector(this);
|
|
|
| if (shouldThrottleRendering()) {
|
| updateViewportIntersectionsForSubtree(std::min(phases, OnlyUpToCompositingCleanPlusScrolling));
|
| @@ -2623,7 +2623,7 @@ void FrameView::updateStyleAndLayoutIfNeededRecursive()
|
|
|
| // FIXME: Calling layout() shouldn't trigger script execution or have any
|
| // observable effects on the frame tree but we're not quite there yet.
|
| - WillBeHeapVector<RefPtrWillBeMember<FrameView>> frameViews;
|
| + HeapVector<Member<FrameView>> frameViews;
|
| for (Frame* child = m_frame->tree().firstChild(); child; child = child->tree().nextSibling()) {
|
| if (!child->isLocalFrame())
|
| continue;
|
| @@ -2946,7 +2946,7 @@ void FrameView::addScrollableArea(ScrollableArea* scrollableArea)
|
| {
|
| ASSERT(scrollableArea);
|
| if (!m_scrollableAreas)
|
| - m_scrollableAreas = adoptPtrWillBeNoop(new ScrollableAreaSet);
|
| + m_scrollableAreas = (new ScrollableAreaSet);
|
| m_scrollableAreas->add(scrollableArea);
|
|
|
| if (ScrollingCoordinator* scrollingCoordinator = this->scrollingCoordinator())
|
| @@ -2967,7 +2967,7 @@ void FrameView::addAnimatingScrollableArea(ScrollableArea* scrollableArea)
|
| {
|
| ASSERT(scrollableArea);
|
| if (!m_animatingScrollableAreas)
|
| - m_animatingScrollableAreas = adoptPtrWillBeNoop(new ScrollableAreaSet);
|
| + m_animatingScrollableAreas = (new ScrollableAreaSet);
|
| m_animatingScrollableAreas->add(scrollableArea);
|
| }
|
|
|
| @@ -3084,7 +3084,7 @@ IntPoint FrameView::maximumScrollPosition() const
|
| return maximumPosition.expandedTo(minimumScrollPosition());
|
| }
|
|
|
| -void FrameView::addChild(PassRefPtrWillBeRawPtr<Widget> prpChild)
|
| +void FrameView::addChild(RawPtr<Widget> prpChild)
|
| {
|
| Widget* child = prpChild.get();
|
| ASSERT(child != this && !child->parent());
|
|
|