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

Unified Diff: third_party/WebKit/Source/core/dom/DocumentOrderedMap.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, 9 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/dom/DocumentOrderedMap.cpp
diff --git a/third_party/WebKit/Source/core/dom/DocumentOrderedMap.cpp b/third_party/WebKit/Source/core/dom/DocumentOrderedMap.cpp
index 0d310c491791aec1205d3ef4e9f0c254a67eee17..aca331db5aede178312b84c277f125d72bfacda9 100644
--- a/third_party/WebKit/Source/core/dom/DocumentOrderedMap.cpp
+++ b/third_party/WebKit/Source/core/dom/DocumentOrderedMap.cpp
@@ -41,7 +41,7 @@ namespace blink {
using namespace HTMLNames;
-PassOwnPtrWillBeRawPtr<DocumentOrderedMap> DocumentOrderedMap::create()
+RawPtr<DocumentOrderedMap> DocumentOrderedMap::create()
{
return adoptPtrWillBeNoop(new DocumentOrderedMap);
}
@@ -50,8 +50,6 @@ DocumentOrderedMap::DocumentOrderedMap()
{
}
-DEFINE_EMPTY_DESTRUCTOR_WILL_BE_REMOVED(DocumentOrderedMap);
-
#if ENABLE(ASSERT)
static int s_removeScopeLevel = 0;
@@ -92,11 +90,11 @@ void DocumentOrderedMap::add(const AtomicString& key, Element* element)
ASSERT(key);
ASSERT(element);
- Map::AddResult addResult = m_map.add(key, adoptPtrWillBeNoop(new MapEntry(element)));
+ Map::AddResult addResult = m_map.add(key, new MapEntry(element));
if (addResult.isNewEntry)
return;
- OwnPtrWillBeMember<MapEntry>& entry = addResult.storedValue->value;
+ Member<MapEntry>& entry = addResult.storedValue->value;
ASSERT(entry->count);
entry->element = nullptr;
entry->count++;
@@ -112,7 +110,7 @@ void DocumentOrderedMap::remove(const AtomicString& key, Element* element)
if (it == m_map.end())
return;
- OwnPtrWillBeMember<MapEntry>& entry = it->value;
+ Member<MapEntry>& entry = it->value;
ASSERT(entry->count);
if (entry->count == 1) {
ASSERT(!entry->element || entry->element == element);
@@ -162,17 +160,17 @@ Element* DocumentOrderedMap::getElementById(const AtomicString& key, const TreeS
return get<keyMatchesId>(key, scope);
}
-const WillBeHeapVector<RawPtrWillBeMember<Element>>& DocumentOrderedMap::getAllElementsById(const AtomicString& key, const TreeScope* scope) const
+const HeapVector<Member<Element>>& DocumentOrderedMap::getAllElementsById(const AtomicString& key, const TreeScope* scope) const
{
ASSERT(key);
ASSERT(scope);
- DEFINE_STATIC_LOCAL(OwnPtrWillBePersistent<WillBeHeapVector<RawPtrWillBeMember<Element>>>, emptyVector, (adoptPtrWillBeNoop(new WillBeHeapVector<RawPtrWillBeMember<Element>>())));
+ DEFINE_STATIC_LOCAL(Persistent<HeapVector<Member<Element>>>, emptyVector, (new HeapVector<Member<Element>>()));
Map::iterator it = m_map.find(key);
if (it == m_map.end())
return *emptyVector;
- OwnPtrWillBeMember<MapEntry>& entry = it->value;
+ Member<MapEntry>& entry = it->value;
ASSERT(entry->count);
if (entry->orderedList.isEmpty()) {
« no previous file with comments | « third_party/WebKit/Source/core/dom/DocumentOrderedMap.h ('k') | third_party/WebKit/Source/core/dom/DocumentParser.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698