Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(206)

Unified Diff: third_party/WebKit/Source/core/frame/FrameView.cpp

Issue 1686483002: Oilpan: Remove most WillBe types from the code base (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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());

Powered by Google App Engine
This is Rietveld 408576698