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

Side by Side Diff: Source/core/dom/Document.h

Issue 16379002: inline WebCore::Node::Node constructor more safely (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: style fixes Created 7 years, 6 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 * (C) 2006 Alexey Proskuryakov (ap@webkit.org) 5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org)
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All r ights reserved. 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All r ights reserved.
7 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/) 7 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/)
8 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies) 8 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
9 * Copyright (C) 2011 Google Inc. All rights reserved. 9 * Copyright (C) 2011 Google Inc. All rights reserved.
10 * 10 *
(...skipping 1471 matching lines...) Expand 10 before | Expand all | Expand 10 after
1482 // This will catch anyone doing an unnecessary cast. 1482 // This will catch anyone doing an unnecessary cast.
1483 void toDocument(const Document*); 1483 void toDocument(const Document*);
1484 1484
1485 // Put these methods here, because they require the Document definition, but we really want to inline them. 1485 // Put these methods here, because they require the Document definition, but we really want to inline them.
1486 1486
1487 inline bool Node::isDocumentNode() const 1487 inline bool Node::isDocumentNode() const
1488 { 1488 {
1489 return this == documentInternal(); 1489 return this == documentInternal();
1490 } 1490 }
1491 1491
1492 inline Node::Node(Document* document, ConstructionType type)
1493 : m_nodeFlags(type)
1494 , m_parentOrShadowHostNode(0)
1495 , m_treeScope(document)
1496 , m_previous(0)
1497 , m_next(0)
1498 {
1499 ScriptWrappable::init(this);
1500 if (!m_treeScope)
1501 m_treeScope = TreeScope::noDocumentInstance();
1502 m_treeScope->guardRef();
1503
1504 #if !defined(NDEBUG) || (defined(DUMP_NODE_STATISTICS) && DUMP_NODE_STATISTICS)
1505 trackForDebugging();
1506 #endif
1507 InspectorCounters::incrementCounter(InspectorCounters::NodeCounter);
1508 }
1509
1510 Node* eventTargetNodeForDocument(Document*); 1492 Node* eventTargetNodeForDocument(Document*);
1511 1493
1512 } // namespace WebCore 1494 } // namespace WebCore
1513 1495
1514 #endif // Document_h 1496 #endif // Document_h
OLDNEW
« no previous file with comments | « Source/core/dom/ContainerNode.h ('k') | Source/core/dom/Node.h » ('j') | Source/core/dom/Text.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698