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

Unified Diff: third_party/WebKit/Source/core/inspector/InspectorDOMAgent.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/inspector/InspectorDOMAgent.cpp
diff --git a/third_party/WebKit/Source/core/inspector/InspectorDOMAgent.cpp b/third_party/WebKit/Source/core/inspector/InspectorDOMAgent.cpp
index c8b35387fb88170c9280470fb92c2c6b4fe0ec50..17d9ff23a6d03c478d4f41b902da22ad803184ef 100644
--- a/third_party/WebKit/Source/core/inspector/InspectorDOMAgent.cpp
+++ b/third_party/WebKit/Source/core/inspector/InspectorDOMAgent.cpp
@@ -164,8 +164,7 @@ v8::Local<v8::Value> nodeV8Value(v8::Local<v8::Context> context, Node* node)
} // namespace
-class InspectorRevalidateDOMTask final : public NoBaseWillBeGarbageCollectedFinalized<InspectorRevalidateDOMTask> {
- USING_FAST_MALLOC_WILL_BE_REMOVED(InspectorRevalidateDOMTask);
+class InspectorRevalidateDOMTask final : public GarbageCollectedFinalized<InspectorRevalidateDOMTask> {
public:
explicit InspectorRevalidateDOMTask(InspectorDOMAgent*);
void scheduleStyleAttrRevalidationFor(Element*);
@@ -174,9 +173,9 @@ public:
DECLARE_TRACE();
private:
- RawPtrWillBeMember<InspectorDOMAgent> m_domAgent;
+ Member<InspectorDOMAgent> m_domAgent;
Timer<InspectorRevalidateDOMTask> m_timer;
- WillBeHeapHashSet<RefPtrWillBeMember<Element> > m_styleAttrInvalidatedElements;
+ HeapHashSet<Member<Element> > m_styleAttrInvalidatedElements;
};
InspectorRevalidateDOMTask::InspectorRevalidateDOMTask(InspectorDOMAgent* domAgent)
@@ -195,7 +194,7 @@ void InspectorRevalidateDOMTask::scheduleStyleAttrRevalidationFor(Element* eleme
void InspectorRevalidateDOMTask::onTimer(Timer<InspectorRevalidateDOMTask>*)
{
// The timer is stopped on m_domAgent destruction, so this method will never be called after m_domAgent has been destroyed.
- WillBeHeapVector<RawPtrWillBeMember<Element> > elements;
+ HeapVector<Member<Element> > elements;
for (auto& attribute : m_styleAttrInvalidatedElements)
elements.append(attribute.get());
m_domAgent->styleAttributeInvalidated(elements);
@@ -276,7 +275,7 @@ InspectorDOMAgent::InspectorDOMAgent(InspectedFrames* inspectedFrames, InjectedS
, m_injectedScriptManager(injectedScriptManager)
, m_client(client)
, m_domListener(nullptr)
- , m_documentNodeToIdMap(adoptPtrWillBeNoop(new NodeToIdMap()))
+ , m_documentNodeToIdMap((new NodeToIdMap()))
, m_lastNodeId(1)
, m_suppressAttributeModifiedEvent(false)
, m_backendNodeIdToInspect(0)
@@ -297,9 +296,9 @@ void InspectorDOMAgent::restore()
innerEnable();
}
-WillBeHeapVector<RawPtrWillBeMember<Document> > InspectorDOMAgent::documents()
+HeapVector<Member<Document> > InspectorDOMAgent::documents()
{
- WillBeHeapVector<RawPtrWillBeMember<Document> > result;
+ HeapVector<Member<Document> > result;
if (m_document) {
for (LocalFrame* frame : *m_inspectedFrames) {
if (Document* document = frame->document())
@@ -509,8 +508,8 @@ Element* InspectorDOMAgent::assertEditableElement(ErrorString* errorString, int
void InspectorDOMAgent::innerEnable()
{
m_state->setBoolean(DOMAgentState::domAgentEnabled, true);
- m_history = adoptPtrWillBeNoop(new InspectorHistory());
- m_domEditor = adoptPtrWillBeNoop(new DOMEditor(m_history.get()));
+ m_history = (new InspectorHistory());
+ m_domEditor = (new DOMEditor(m_history.get()));
m_document = m_inspectedFrames->root()->document();
m_instrumentingAgents->setInspectorDOMAgent(this);
if (m_backendNodeIdToInspect)
@@ -608,7 +607,7 @@ Node* InspectorDOMAgent::nodeForId(int id)
if (!id)
return nullptr;
- WillBeHeapHashMap<int, RawPtrWillBeMember<Node> >::iterator it = m_idToNode.find(id);
+ HeapHashMap<int, Member<Node> >::iterator it = m_idToNode.find(id);
if (it != m_idToNode.end())
return it->value;
return nullptr;
@@ -640,7 +639,7 @@ void InspectorDOMAgent::querySelector(ErrorString* errorString, int nodeId, cons
return;
TrackExceptionState exceptionState;
- RefPtrWillBeRawPtr<Element> element = toContainerNode(node)->querySelector(AtomicString(selectors), exceptionState);
+ RawPtr<Element> element = toContainerNode(node)->querySelector(AtomicString(selectors), exceptionState);
if (exceptionState.hadException()) {
*errorString = "DOM Error while querying";
return;
@@ -657,7 +656,7 @@ void InspectorDOMAgent::querySelectorAll(ErrorString* errorString, int nodeId, c
return;
TrackExceptionState exceptionState;
- RefPtrWillBeRawPtr<StaticElementList> elements = toContainerNode(node)->querySelectorAll(AtomicString(selectors), exceptionState);
+ RawPtr<StaticElementList> elements = toContainerNode(node)->querySelectorAll(AtomicString(selectors), exceptionState);
if (exceptionState.hadException()) {
*errorString = "DOM Error while querying";
return;
@@ -684,7 +683,7 @@ int InspectorDOMAgent::pushNodePathToFrontend(Node* nodeToPush, NodeToIdMap* nod
return result;
Node* node = nodeToPush;
- WillBeHeapVector<RawPtrWillBeMember<Node> > path;
+ HeapVector<Member<Node> > path;
while (true) {
Node* parent = innerParentNode(node);
@@ -718,7 +717,7 @@ int InspectorDOMAgent::pushNodePathToFrontend(Node* nodeToPush)
node = parent;
// Node being pushed is detached -> push subtree root.
- OwnPtrWillBeRawPtr<NodeToIdMap> newMap = adoptPtrWillBeNoop(new NodeToIdMap);
+ RawPtr<NodeToIdMap> newMap = (new NodeToIdMap);
NodeToIdMap* danglingMap = newMap.get();
m_danglingNodeToIdMaps.append(newMap.release());
RefPtr<TypeBuilder::Array<TypeBuilder::DOM::Node> > children = TypeBuilder::Array<TypeBuilder::DOM::Node>::create();
@@ -749,7 +748,7 @@ void InspectorDOMAgent::setAttributesAsText(ErrorString* errorString, int elemen
return;
String markup = "<span " + text + "></span>";
- RefPtrWillBeRawPtr<DocumentFragment> fragment = element->document().createDocumentFragment();
+ RawPtr<DocumentFragment> fragment = element->document().createDocumentFragment();
bool shouldIgnoreCase = element->document().isHTMLDocument() && element->isHTMLElement();
// Not all elements can represent the context (i.e. IFRAME), hence using document.body.
@@ -820,7 +819,7 @@ void InspectorDOMAgent::setNodeName(ErrorString* errorString, int nodeId, const
return;
TrackExceptionState exceptionState;
- RefPtrWillBeRawPtr<Element> newElem = oldNode->document().createElement(AtomicString(tagName), exceptionState);
+ RawPtr<Element> newElem = oldNode->document().createElement(AtomicString(tagName), exceptionState);
if (exceptionState.hadException())
return;
@@ -967,8 +966,8 @@ void InspectorDOMAgent::performSearch(ErrorString*, const String& whitespaceTrim
if (endQuoteFound)
attributeQuery = attributeQuery.left(attributeQuery.length() - 1);
- WillBeHeapVector<RawPtrWillBeMember<Document> > docs = documents();
- WillBeHeapListHashSet<RawPtrWillBeMember<Node> > resultCollector;
+ HeapVector<Member<Document> > docs = documents();
+ HeapListHashSet<Member<Node> > resultCollector;
for (Document* document : docs) {
Node* documentElement = document->documentElement();
@@ -1042,7 +1041,7 @@ void InspectorDOMAgent::performSearch(ErrorString*, const String& whitespaceTrim
// Selector evaluation
for (Document* document : docs) {
TrackExceptionState exceptionState;
- RefPtrWillBeRawPtr<StaticElementList> elementList = document->querySelectorAll(AtomicString(whitespaceTrimmedQuery), exceptionState);
+ RawPtr<StaticElementList> elementList = document->querySelectorAll(AtomicString(whitespaceTrimmedQuery), exceptionState);
if (exceptionState.hadException() || !elementList)
continue;
@@ -1053,7 +1052,7 @@ void InspectorDOMAgent::performSearch(ErrorString*, const String& whitespaceTrim
}
*searchId = IdentifiersFactory::createIdentifier();
- WillBeHeapVector<RefPtrWillBeMember<Node> >* resultsIt = &m_searchResults.add(*searchId, WillBeHeapVector<RefPtrWillBeMember<Node> >()).storedValue->value;
+ HeapVector<Member<Node> >* resultsIt = &m_searchResults.add(*searchId, HeapVector<Member<Node> >()).storedValue->value;
for (auto& result : resultCollector)
resultsIt->append(result);
@@ -1302,7 +1301,7 @@ void InspectorDOMAgent::copyTo(ErrorString* errorString, int nodeId, int targetE
}
// The clone is deep by default.
- RefPtrWillBeRawPtr<Node> clonedNode = node->cloneNode(true);
+ RawPtr<Node> clonedNode = node->cloneNode(true);
if (!clonedNode) {
*errorString = "Failed to clone node";
return;
@@ -1857,7 +1856,7 @@ void InspectorDOMAgent::didRemoveDOMAttr(Element* element, const QualifiedName&
frontend()->attributeRemoved(id, name.toString());
}
-void InspectorDOMAgent::styleAttributeInvalidated(const WillBeHeapVector<RawPtrWillBeMember<Element> >& elements)
+void InspectorDOMAgent::styleAttributeInvalidated(const HeapVector<Member<Element> >& elements)
{
RefPtr<TypeBuilder::Array<int> > nodeIds = TypeBuilder::Array<int>::create();
for (unsigned i = 0, size = elements.size(); i < size; ++i) {
@@ -1885,10 +1884,10 @@ void InspectorDOMAgent::characterDataModified(CharacterData* characterData)
frontend()->characterDataModified(id, characterData->data());
}
-RawPtrWillBeMember<InspectorRevalidateDOMTask> InspectorDOMAgent::revalidateTask()
+Member<InspectorRevalidateDOMTask> InspectorDOMAgent::revalidateTask()
{
if (!m_revalidateTask)
- m_revalidateTask = adoptPtrWillBeNoop(new InspectorRevalidateDOMTask(this));
+ m_revalidateTask = (new InspectorRevalidateDOMTask(this));
return m_revalidateTask.get();
}
@@ -1933,7 +1932,7 @@ void InspectorDOMAgent::didPerformElementShadowDistribution(Element* shadowHost)
return;
for (ShadowRoot* root = shadowHost->youngestShadowRoot(); root; root = root->olderShadowRoot()) {
- const WillBeHeapVector<RefPtrWillBeMember<InsertionPoint>>& insertionPoints = root->descendantInsertionPoints();
+ const HeapVector<Member<InsertionPoint>>& insertionPoints = root->descendantInsertionPoints();
for (const auto& it : insertionPoints) {
InsertionPoint* insertionPoint = it.get();
int insertionPointId = m_documentNodeToIdMap->get(insertionPoint);

Powered by Google App Engine
This is Rietveld 408576698