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

Side by Side Diff: Source/core/frame/EventHandlerRegistry.cpp

Issue 1166623002: Oilpan: Remove a visitor parameter from isHeapObjectAlive (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 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/css/resolver/MatchedPropertiesCache.h ('k') | Source/core/frame/LocalFrame.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "config.h" 5 #include "config.h"
6 #include "core/frame/EventHandlerRegistry.h" 6 #include "core/frame/EventHandlerRegistry.h"
7 7
8 #include "core/events/ThreadLocalEventNames.h" 8 #include "core/events/ThreadLocalEventNames.h"
9 #include "core/frame/LocalDOMWindow.h" 9 #include "core/frame/LocalDOMWindow.h"
10 #include "core/frame/LocalFrame.h" 10 #include "core/frame/LocalFrame.h"
(...skipping 191 matching lines...) Expand 10 before | Expand all | Expand 10 after
202 202
203 void EventHandlerRegistry::clearWeakMembers(Visitor* visitor) 203 void EventHandlerRegistry::clearWeakMembers(Visitor* visitor)
204 { 204 {
205 Vector<EventTarget*> deadTargets; 205 Vector<EventTarget*> deadTargets;
206 for (size_t i = 0; i < EventHandlerClassCount; ++i) { 206 for (size_t i = 0; i < EventHandlerClassCount; ++i) {
207 EventHandlerClass handlerClass = static_cast<EventHandlerClass>(i); 207 EventHandlerClass handlerClass = static_cast<EventHandlerClass>(i);
208 const EventTargetSet* targets = &m_targets[handlerClass]; 208 const EventTargetSet* targets = &m_targets[handlerClass];
209 for (const auto& eventTarget : *targets) { 209 for (const auto& eventTarget : *targets) {
210 Node* node = eventTarget.key->toNode(); 210 Node* node = eventTarget.key->toNode();
211 LocalDOMWindow* window = eventTarget.key->toDOMWindow(); 211 LocalDOMWindow* window = eventTarget.key->toDOMWindow();
212 if (node && !visitor->isHeapObjectAlive(node)) { 212 if (node && !Heap::isHeapObjectAlive(node)) {
213 deadTargets.append(node); 213 deadTargets.append(node);
214 } else if (window && !visitor->isHeapObjectAlive(window)) { 214 } else if (window && !Heap::isHeapObjectAlive(window)) {
215 deadTargets.append(window); 215 deadTargets.append(window);
216 } 216 }
217 } 217 }
218 } 218 }
219 for (size_t i = 0; i < deadTargets.size(); ++i) 219 for (size_t i = 0; i < deadTargets.size(); ++i)
220 didRemoveAllEventHandlers(*deadTargets[i]); 220 didRemoveAllEventHandlers(*deadTargets[i]);
221 } 221 }
222 222
223 void EventHandlerRegistry::documentDetached(Document& document) 223 void EventHandlerRegistry::documentDetached(Document& document)
224 { 224 {
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
264 ASSERT(window->frame()); 264 ASSERT(window->frame());
265 ASSERT(window->frame()->host()); 265 ASSERT(window->frame()->host());
266 ASSERT(window->frame()->host() == &m_frameHost); 266 ASSERT(window->frame()->host() == &m_frameHost);
267 } 267 }
268 } 268 }
269 } 269 }
270 #endif // ENABLE(ASSERT) 270 #endif // ENABLE(ASSERT)
271 } 271 }
272 272
273 } // namespace blink 273 } // namespace blink
OLDNEW
« no previous file with comments | « Source/core/css/resolver/MatchedPropertiesCache.h ('k') | Source/core/frame/LocalFrame.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698