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

Side by Side Diff: Source/core/dom/Node.cpp

Issue 280123002: Oilpan: move LiveNodeList collections to the heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Pre-emptively GC a long runnning test Created 6 years, 7 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2001 Dirk Mueller (mueller@kde.org) 4 * (C) 2001 Dirk Mueller (mueller@kde.org)
5 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All r ights reserved. 5 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All r ights reserved.
6 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies) 6 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
7 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmo bile.com/) 7 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmo bile.com/)
8 * 8 *
9 * This library is free software; you can redistribute it and/or 9 * This library is free software; you can redistribute it and/or
10 * modify it under the terms of the GNU Library General Public 10 * modify it under the terms of the GNU Library General Public
(...skipping 357 matching lines...) Expand 10 before | Expand all | Expand 10 after
368 String Node::nodeValue() const 368 String Node::nodeValue() const
369 { 369 {
370 return String(); 370 return String();
371 } 371 }
372 372
373 void Node::setNodeValue(const String&) 373 void Node::setNodeValue(const String&)
374 { 374 {
375 // By default, setting nodeValue has no effect. 375 // By default, setting nodeValue has no effect.
376 } 376 }
377 377
378 PassRefPtr<NodeList> Node::childNodes() 378 PassRefPtrWillBeRawPtr<NodeList> Node::childNodes()
379 { 379 {
380 if (isContainerNode()) 380 if (isContainerNode())
381 return ensureRareData().ensureNodeLists().ensureChildNodeList(toContaine rNode(*this)); 381 return ensureRareData().ensureNodeLists().ensureChildNodeList(toContaine rNode(*this));
382 return ensureRareData().ensureNodeLists().ensureEmptyChildNodeList(*this); 382 return ensureRareData().ensureNodeLists().ensureEmptyChildNodeList(*this);
383 } 383 }
384 384
385 Node& Node::lastDescendantOrSelf() const 385 Node& Node::lastDescendantOrSelf() const
386 { 386 {
387 Node* n = const_cast<Node*>(this); 387 Node* n = const_cast<Node*>(this);
388 while (n && n->lastChild()) 388 while (n && n->lastChild())
(...skipping 2068 matching lines...) Expand 10 before | Expand all | Expand 10 after
2457 { 2457 {
2458 unsigned count = connectedSubframeCount(); 2458 unsigned count = connectedSubframeCount();
2459 2459
2460 if (!count) 2460 if (!count)
2461 return; 2461 return;
2462 2462
2463 for (Node* node = parentOrShadowHostNode(); node; node = node->parentOrShado wHostNode()) 2463 for (Node* node = parentOrShadowHostNode(); node; node = node->parentOrShado wHostNode())
2464 node->incrementConnectedSubframeCount(count); 2464 node->incrementConnectedSubframeCount(count);
2465 } 2465 }
2466 2466
2467 PassRefPtr<NodeList> Node::getDestinationInsertionPoints() 2467 PassRefPtrWillBeRawPtr<NodeList> Node::getDestinationInsertionPoints()
2468 { 2468 {
2469 document().updateDistributionForNodeIfNeeded(this); 2469 document().updateDistributionForNodeIfNeeded(this);
2470 Vector<InsertionPoint*, 8> insertionPoints; 2470 Vector<InsertionPoint*, 8> insertionPoints;
2471 collectDestinationInsertionPoints(*this, insertionPoints); 2471 collectDestinationInsertionPoints(*this, insertionPoints);
2472 Vector<RefPtr<Node> > filteredInsertionPoints; 2472 Vector<RefPtr<Node> > filteredInsertionPoints;
2473 for (size_t i = 0; i < insertionPoints.size(); ++i) { 2473 for (size_t i = 0; i < insertionPoints.size(); ++i) {
2474 InsertionPoint* insertionPoint = insertionPoints[i]; 2474 InsertionPoint* insertionPoint = insertionPoints[i];
2475 ASSERT(insertionPoint->containingShadowRoot()); 2475 ASSERT(insertionPoint->containingShadowRoot());
2476 if (insertionPoint->containingShadowRoot()->type() != ShadowRoot::UserAg entShadowRoot) 2476 if (insertionPoint->containingShadowRoot()->type() != ShadowRoot::UserAg entShadowRoot)
2477 filteredInsertionPoints.append(insertionPoint); 2477 filteredInsertionPoints.append(insertionPoint);
(...skipping 115 matching lines...) Expand 10 before | Expand all | Expand 10 after
2593 node->showTreeForThis(); 2593 node->showTreeForThis();
2594 } 2594 }
2595 2595
2596 void showNodePath(const WebCore::Node* node) 2596 void showNodePath(const WebCore::Node* node)
2597 { 2597 {
2598 if (node) 2598 if (node)
2599 node->showNodePathForThis(); 2599 node->showNodePathForThis();
2600 } 2600 }
2601 2601
2602 #endif 2602 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698