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

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

Issue 2657443005: Migrate WTF::HashSet::add() to ::insert() [part 1 of N] (Closed)
Patch Set: Created 3 years, 11 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 9fca55f4f81012ca7107e2c2842cefaaf4385a0d..7a28a110516b2dd411872bab4bb947ff2dde40fb 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -1438,7 +1438,7 @@ LayoutReplaced* FrameView::embeddedReplacedContent() const {
}
void FrameView::addPart(LayoutPart* object) {
- m_parts.add(object);
+ m_parts.insert(object);
}
void FrameView::removePart(LayoutPart* object) {
@@ -1477,7 +1477,7 @@ void FrameView::addPartToUpdate(LayoutEmbeddedObject& object) {
if (isHTMLObjectElement(*node) || isHTMLEmbedElement(*node))
toHTMLPlugInElement(node)->setNeedsWidgetUpdate(true);
- m_partUpdateSet.add(&object);
+ m_partUpdateSet.insert(&object);
}
void FrameView::setDisplayMode(WebDisplayMode mode) {
@@ -1539,7 +1539,7 @@ bool FrameView::contentsInCompositedLayer() const {
void FrameView::addBackgroundAttachmentFixedObject(LayoutObject* object) {
ASSERT(!m_backgroundAttachmentFixedObjects.contains(object));
- m_backgroundAttachmentFixedObjects.add(object);
+ m_backgroundAttachmentFixedObjects.insert(object);
if (ScrollingCoordinator* scrollingCoordinator = this->scrollingCoordinator())
scrollingCoordinator->frameViewHasBackgroundAttachmentFixedObjectsDidChange(
this);
@@ -1575,7 +1575,7 @@ void FrameView::addViewportConstrainedObject(LayoutObject* object) {
}
if (!m_viewportConstrainedObjects->contains(object)) {
- m_viewportConstrainedObjects->add(object);
+ m_viewportConstrainedObjects->insert(object);
if (ScrollingCoordinator* scrollingCoordinator =
this->scrollingCoordinator())
@@ -3599,7 +3599,7 @@ std::unique_ptr<JSONArray> FrameView::trackedObjectPaintInvalidationsAsJSON()
void FrameView::addResizerArea(LayoutBox& resizerBox) {
if (!m_resizerAreas)
m_resizerAreas = WTF::wrapUnique(new ResizerAreaSet);
- m_resizerAreas->add(&resizerBox);
+ m_resizerAreas->insert(&resizerBox);
}
void FrameView::removeResizerArea(LayoutBox& resizerBox) {
@@ -3615,7 +3615,7 @@ void FrameView::addScrollableArea(ScrollableArea* scrollableArea) {
ASSERT(scrollableArea);
if (!m_scrollableAreas)
m_scrollableAreas = new ScrollableAreaSet;
- m_scrollableAreas->add(scrollableArea);
+ m_scrollableAreas->insert(scrollableArea);
if (ScrollingCoordinator* scrollingCoordinator = this->scrollingCoordinator())
scrollingCoordinator->scrollableAreasDidChange();
@@ -3634,7 +3634,7 @@ void FrameView::addAnimatingScrollableArea(ScrollableArea* scrollableArea) {
ASSERT(scrollableArea);
if (!m_animatingScrollableAreas)
m_animatingScrollableAreas = new ScrollableAreaSet;
- m_animatingScrollableAreas->add(scrollableArea);
+ m_animatingScrollableAreas->insert(scrollableArea);
}
void FrameView::removeAnimatingScrollableArea(ScrollableArea* scrollableArea) {
@@ -3759,7 +3759,7 @@ IntSize FrameView::maximumScrollOffsetInt() const {
void FrameView::addChild(Widget* child) {
ASSERT(child != this && !child->parent());
child->setParent(this);
- m_children.add(child);
+ m_children.insert(child);
}
void FrameView::setScrollbarModes(ScrollbarMode horizontalMode,
« no previous file with comments | « third_party/WebKit/Source/core/frame/FrameSerializer.cpp ('k') | third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698