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

Side by Side Diff: Source/core/editing/WrapContentsInDummySpanCommand.cpp

Issue 435733002: Have getChildNodes() take a ContainerNode in argument (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 4 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2005, 2008 Apple Inc. All rights reserved. 2 * Copyright (C) 2005, 2008 Apple Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 23 matching lines...) Expand all
34 34
35 WrapContentsInDummySpanCommand::WrapContentsInDummySpanCommand(PassRefPtrWillBeR awPtr<Element> element) 35 WrapContentsInDummySpanCommand::WrapContentsInDummySpanCommand(PassRefPtrWillBeR awPtr<Element> element)
36 : SimpleEditCommand(element->document()) 36 : SimpleEditCommand(element->document())
37 , m_element(element) 37 , m_element(element)
38 { 38 {
39 ASSERT(m_element); 39 ASSERT(m_element);
40 } 40 }
41 41
42 void WrapContentsInDummySpanCommand::executeApply() 42 void WrapContentsInDummySpanCommand::executeApply()
43 { 43 {
44 WillBeHeapVector<RefPtrWillBeMember<Node> > children; 44 NodeVector children;
45 for (Node* child = m_element->firstChild(); child; child = child->nextSiblin g()) 45 getChildNodes(*m_element, children);
46 children.append(child);
47 46
48 size_t size = children.size(); 47 size_t size = children.size();
49 for (size_t i = 0; i < size; ++i) 48 for (size_t i = 0; i < size; ++i)
50 m_dummySpan->appendChild(children[i].release(), IGNORE_EXCEPTION); 49 m_dummySpan->appendChild(children[i].release(), IGNORE_EXCEPTION);
51 50
52 m_element->appendChild(m_dummySpan.get(), IGNORE_EXCEPTION); 51 m_element->appendChild(m_dummySpan.get(), IGNORE_EXCEPTION);
53 } 52 }
54 53
55 void WrapContentsInDummySpanCommand::doApply() 54 void WrapContentsInDummySpanCommand::doApply()
56 { 55 {
57 m_dummySpan = createStyleSpanElement(document()); 56 m_dummySpan = createStyleSpanElement(document());
58 57
59 executeApply(); 58 executeApply();
60 } 59 }
61 60
62 void WrapContentsInDummySpanCommand::doUnapply() 61 void WrapContentsInDummySpanCommand::doUnapply()
63 { 62 {
64 ASSERT(m_element); 63 ASSERT(m_element);
65 64
66 if (!m_dummySpan || !m_element->hasEditableStyle()) 65 if (!m_dummySpan || !m_element->hasEditableStyle())
67 return; 66 return;
68 67
69 WillBeHeapVector<RefPtrWillBeMember<Node> > children; 68 NodeVector children;
70 for (Node* child = m_dummySpan->firstChild(); child; child = child->nextSibl ing()) 69 getChildNodes(*m_dummySpan, children);
71 children.append(child);
72 70
73 size_t size = children.size(); 71 size_t size = children.size();
74 for (size_t i = 0; i < size; ++i) 72 for (size_t i = 0; i < size; ++i)
75 m_element->appendChild(children[i].release(), IGNORE_EXCEPTION); 73 m_element->appendChild(children[i].release(), IGNORE_EXCEPTION);
76 74
77 m_dummySpan->remove(IGNORE_EXCEPTION); 75 m_dummySpan->remove(IGNORE_EXCEPTION);
78 } 76 }
79 77
80 void WrapContentsInDummySpanCommand::doReapply() 78 void WrapContentsInDummySpanCommand::doReapply()
81 { 79 {
82 ASSERT(m_element); 80 ASSERT(m_element);
83 81
84 if (!m_dummySpan || !m_element->hasEditableStyle()) 82 if (!m_dummySpan || !m_element->hasEditableStyle())
85 return; 83 return;
86 84
87 executeApply(); 85 executeApply();
88 } 86 }
89 87
90 void WrapContentsInDummySpanCommand::trace(Visitor* visitor) 88 void WrapContentsInDummySpanCommand::trace(Visitor* visitor)
91 { 89 {
92 visitor->trace(m_element); 90 visitor->trace(m_element);
93 visitor->trace(m_dummySpan); 91 visitor->trace(m_dummySpan);
94 SimpleEditCommand::trace(visitor); 92 SimpleEditCommand::trace(visitor);
95 } 93 }
96 94
97 } // namespace blink 95 } // namespace blink
OLDNEW
« Source/core/editing/ApplyStyleCommand.cpp ('K') | « Source/core/editing/SplitElementCommand.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698