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

Unified Diff: third_party/WebKit/Source/core/layout/TextAutosizer.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/layout/TextAutosizer.cpp
diff --git a/third_party/WebKit/Source/core/layout/TextAutosizer.cpp b/third_party/WebKit/Source/core/layout/TextAutosizer.cpp
index 361403ecfc3e5381f879b73d4607d410aa2eef5b..a684c82c109577d5a75b6fea910debf71666963c 100644
--- a/third_party/WebKit/Source/core/layout/TextAutosizer.cpp
+++ b/third_party/WebKit/Source/core/layout/TextAutosizer.cpp
@@ -361,7 +361,7 @@ void TextAutosizer::destroy(LayoutBlock* block) {
TextAutosizer::BeginLayoutBehavior TextAutosizer::prepareForLayout(
LayoutBlock* block) {
#if DCHECK_IS_ON()
- m_blocksThatHaveBegunLayout.add(block);
+ m_blocksThatHaveBegunLayout.insert(block);
#endif
if (!m_firstBlockToBeginLayout) {
@@ -386,7 +386,7 @@ void TextAutosizer::prepareClusterStack(LayoutObject* layoutObject) {
if (layoutObject->isLayoutBlock()) {
LayoutBlock* block = toLayoutBlock(layoutObject);
#if DCHECK_IS_ON()
- m_blocksThatHaveBegunLayout.add(block);
+ m_blocksThatHaveBegunLayout.insert(block);
#endif
if (Cluster* cluster = maybeCreateCluster(block))
m_clusterStack.push_back(WTF::wrapUnique(cluster));
@@ -556,8 +556,8 @@ void TextAutosizer::markSuperclusterForConsistencyCheck(LayoutObject* object) {
if (supercluster &&
supercluster->m_inheritParentMultiplier == DontInheritMultiplier) {
if (supercluster->m_hasEnoughTextToAutosize == NotEnoughText) {
- m_fingerprintMapper.getPotentiallyInconsistentSuperclusters().add(
- supercluster);
+ m_fingerprintMapper.getPotentiallyInconsistentSuperclusters()
+ .insert(supercluster);
}
return;
}
@@ -572,7 +572,7 @@ void TextAutosizer::markSuperclusterForConsistencyCheck(LayoutObject* object) {
// If we didn't add any supercluster, we should add one.
if (lastSupercluster) {
- m_fingerprintMapper.getPotentiallyInconsistentSuperclusters().add(
+ m_fingerprintMapper.getPotentiallyInconsistentSuperclusters().insert(
lastSupercluster);
}
}
@@ -1278,7 +1278,7 @@ void TextAutosizer::FingerprintMapper::addTentativeClusterRoot(
m_blocksForFingerprint.add(fingerprint, std::unique_ptr<BlockSet>());
if (addResult.isNewEntry)
addResult.storedValue->value = WTF::wrapUnique(new BlockSet);
- addResult.storedValue->value->add(block);
+ addResult.storedValue->value->insert(block);
#if DCHECK_IS_ON()
assertMapsAreConsistent();
#endif

Powered by Google App Engine
This is Rietveld 408576698