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

Unified Diff: third_party/WebKit/Source/core/svg/animation/SVGSMILElement.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/svg/animation/SVGSMILElement.cpp
diff --git a/third_party/WebKit/Source/core/svg/animation/SVGSMILElement.cpp b/third_party/WebKit/Source/core/svg/animation/SVGSMILElement.cpp
index 42c9c97d87b74a2e1a6920aee4b97e68c6b7608f..c9787a81bd85d97b32616e599fc1bae9034492ab 100644
--- a/third_party/WebKit/Source/core/svg/animation/SVGSMILElement.cpp
+++ b/third_party/WebKit/Source/core/svg/animation/SVGSMILElement.cpp
@@ -430,7 +430,7 @@ void SVGSMILElement::parseBeginOrEnd(const String& parseString,
HashSet<SMILTime> existing;
for (unsigned n = 0; n < timeList.size(); ++n) {
if (!timeList[n].time().isUnresolved())
- existing.add(timeList[n].time().value());
+ existing.insert(timeList[n].time().value());
}
Vector<String> splitString;
parseString.split(';', splitString);
@@ -1178,7 +1178,7 @@ void SVGSMILElement::notifyDependentsIntervalChanged() {
// use a HashSet of untraced heap references -- any conservative GC which
// strikes before unwinding will find these elements on the stack.
DEFINE_STATIC_LOCAL(HashSet<UntracedMember<SVGSMILElement>>, loopBreaker, ());
- if (!loopBreaker.add(this).isNewEntry)
+ if (!loopBreaker.insert(this).isNewEntry)
return;
for (SVGSMILElement* element : m_syncBaseDependents)
@@ -1216,7 +1216,7 @@ void SVGSMILElement::createInstanceTimesFromSyncbase(SVGSMILElement* syncBase) {
}
void SVGSMILElement::addSyncBaseDependent(SVGSMILElement* animation) {
- m_syncBaseDependents.add(animation);
+ m_syncBaseDependents.insert(animation);
if (m_interval.begin.isFinite())
animation->createInstanceTimesFromSyncbase(this);
}

Powered by Google App Engine
This is Rietveld 408576698