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

Side by Side Diff: Source/core/events/Event.cpp

Issue 334713006: Use stricter typing for NodeLists throughout the code base (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase Created 6 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 | Annotate | Revision Log
« no previous file with comments | « Source/core/events/Event.h ('k') | Source/core/events/NodeEventContext.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) 2001 Peter Kelly (pmk@post.com) 2 * Copyright (C) 2001 Peter Kelly (pmk@post.com)
3 * Copyright (C) 2001 Tobias Anton (anton@stud.fbi.fh-darmstadt.de) 3 * Copyright (C) 2001 Tobias Anton (anton@stud.fbi.fh-darmstadt.de)
4 * Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com) 4 * Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
5 * Copyright (C) 2003, 2005, 2006, 2008 Apple Inc. All rights reserved. 5 * Copyright (C) 2003, 2005, 2006, 2008 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 199 matching lines...) Expand 10 before | Expand all | Expand 10 after
210 m_underlyingEvent = ue; 210 m_underlyingEvent = ue;
211 } 211 }
212 212
213 EventPath& Event::ensureEventPath() 213 EventPath& Event::ensureEventPath()
214 { 214 {
215 if (!m_eventPath) 215 if (!m_eventPath)
216 m_eventPath = adoptPtrWillBeNoop(new EventPath(this)); 216 m_eventPath = adoptPtrWillBeNoop(new EventPath(this));
217 return *m_eventPath; 217 return *m_eventPath;
218 } 218 }
219 219
220 PassRefPtrWillBeRawPtr<NodeList> Event::path() const 220 PassRefPtrWillBeRawPtr<StaticNodeList> Event::path() const
221 { 221 {
222 if (!m_currentTarget || !m_currentTarget->toNode()) 222 if (!m_currentTarget || !m_currentTarget->toNode())
223 return StaticNodeList::createEmpty(); 223 return StaticNodeList::createEmpty();
224 Node* node = m_currentTarget->toNode(); 224 Node* node = m_currentTarget->toNode();
225 // FIXME: Support SVG Elements. 225 // FIXME: Support SVG Elements.
226 if (node->isSVGElement()) 226 if (node->isSVGElement())
227 return StaticNodeList::createEmpty(); 227 return StaticNodeList::createEmpty();
228 size_t eventPathSize = m_eventPath->size(); 228 size_t eventPathSize = m_eventPath->size();
229 for (size_t i = 0; i < eventPathSize; ++i) { 229 for (size_t i = 0; i < eventPathSize; ++i) {
230 if (node == (*m_eventPath)[i].node()) { 230 if (node == (*m_eventPath)[i].node()) {
(...skipping 17 matching lines...) Expand all
248 248
249 void Event::trace(Visitor* visitor) 249 void Event::trace(Visitor* visitor)
250 { 250 {
251 visitor->trace(m_currentTarget); 251 visitor->trace(m_currentTarget);
252 visitor->trace(m_target); 252 visitor->trace(m_target);
253 visitor->trace(m_underlyingEvent); 253 visitor->trace(m_underlyingEvent);
254 visitor->trace(m_eventPath); 254 visitor->trace(m_eventPath);
255 } 255 }
256 256
257 } // namespace WebCore 257 } // namespace WebCore
OLDNEW
« no previous file with comments | « Source/core/events/Event.h ('k') | Source/core/events/NodeEventContext.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698