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

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

Issue 313023004: Oilpan: Replace RefPtr to Node in MutationObserver-related classes with Oilpan transition types. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: 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 | « no previous file | Source/core/dom/MutationObserverRegistration.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 /* 1 /*
2 * Copyright (C) 2011 Google Inc. All rights reserved. 2 * Copyright (C) 2011 Google 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 are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 void trace(Visitor*); 64 void trace(Visitor*);
65 65
66 void dispose(); 66 void dispose();
67 67
68 private: 68 private:
69 MutationObserverRegistration(MutationObserver&, Node*, MutationObserverOptio ns, const HashSet<AtomicString>& attributeFilter); 69 MutationObserverRegistration(MutationObserver&, Node*, MutationObserverOptio ns, const HashSet<AtomicString>& attributeFilter);
70 70
71 RefPtrWillBeMember<MutationObserver> m_observer; 71 RefPtrWillBeMember<MutationObserver> m_observer;
72 RawPtrWillBeWeakMember<Node> m_registrationNode; 72 RawPtrWillBeWeakMember<Node> m_registrationNode;
73 RefPtrWillBeMember<Node> m_registrationNodeKeepAlive; 73 RefPtrWillBeMember<Node> m_registrationNodeKeepAlive;
74 typedef HashSet<RefPtr<Node> > NodeHashSet; 74 typedef WillBeHeapHashSet<RefPtrWillBeMember<Node> > NodeHashSet;
75 OwnPtr<NodeHashSet> m_transientRegistrationNodes; 75 OwnPtr<NodeHashSet> m_transientRegistrationNodes;
haraken 2014/06/04 07:25:33 This should be OwnPtrWillBeMember<NodeHashSet>. O
sof 2014/06/04 07:26:22 If we want to lazily instantiate the hash set, cou
zerny-chromium 2014/06/04 07:45:59 I'm surprised that this does not trigger a compile
tkent 2014/06/04 09:20:23 Thank you for pointing this out. Yeah, I missed it
76 76
77 MutationObserverOptions m_options; 77 MutationObserverOptions m_options;
78 HashSet<AtomicString> m_attributeFilter; 78 HashSet<AtomicString> m_attributeFilter;
79 }; 79 };
80 80
81 } // namespace WebCore 81 } // namespace WebCore
82 82
83 #endif // MutationObserverRegistration_h 83 #endif // MutationObserverRegistration_h
OLDNEW
« no previous file with comments | « no previous file | Source/core/dom/MutationObserverRegistration.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698