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

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

Issue 23819007: Have Node::document() return a reference instead of a pointer (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase on master Created 7 years, 3 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 | Annotate | Revision Log
« no previous file with comments | « Source/core/dom/DocumentStyleSheetCollection.cpp ('k') | Source/core/dom/Element.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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, 2008, 2009 Apple Inc. All rights reserved. 5 * Copyright (C) 2004, 2005, 2006, 2008, 2009 Apple Inc. All rights reserved.
6 * 6 *
7 * This library is free software; you can redistribute it and/or 7 * This library is free software; you can redistribute it and/or
8 * modify it under the terms of the GNU Library General Public 8 * modify it under the terms of the GNU Library General Public
9 * License as published by the Free Software Foundation; either 9 * License as published by the Free Software Foundation; either
10 * version 2 of the License, or (at your option) any later version. 10 * version 2 of the License, or (at your option) any later version.
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 return name(); 47 return name();
48 } 48 }
49 49
50 Node::NodeType DocumentType::nodeType() const 50 Node::NodeType DocumentType::nodeType() const
51 { 51 {
52 return DOCUMENT_TYPE_NODE; 52 return DOCUMENT_TYPE_NODE;
53 } 53 }
54 54
55 PassRefPtr<Node> DocumentType::cloneNode(bool /*deep*/) 55 PassRefPtr<Node> DocumentType::cloneNode(bool /*deep*/)
56 { 56 {
57 return create(document(), m_name, m_publicId, m_systemId); 57 return create(&document(), m_name, m_publicId, m_systemId);
58 } 58 }
59 59
60 Node::InsertionNotificationRequest DocumentType::insertedInto(ContainerNode* ins ertionPoint) 60 Node::InsertionNotificationRequest DocumentType::insertedInto(ContainerNode* ins ertionPoint)
61 { 61 {
62 Node::insertedInto(insertionPoint); 62 Node::insertedInto(insertionPoint);
63 63
64 // DocumentType can only be inserted into a Document. 64 // DocumentType can only be inserted into a Document.
65 ASSERT(parentNode()->isDocumentNode()); 65 ASSERT(parentNode()->isDocumentNode());
66 66
67 document()->setDoctype(this); 67 document().setDoctype(this);
68 68
69 return InsertionDone; 69 return InsertionDone;
70 } 70 }
71 71
72 void DocumentType::removedFrom(ContainerNode* insertionPoint) 72 void DocumentType::removedFrom(ContainerNode* insertionPoint)
73 { 73 {
74 document()->setDoctype(0); 74 document().setDoctype(0);
75 Node::removedFrom(insertionPoint); 75 Node::removedFrom(insertionPoint);
76 } 76 }
77 77
78 } 78 }
OLDNEW
« no previous file with comments | « Source/core/dom/DocumentStyleSheetCollection.cpp ('k') | Source/core/dom/Element.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698