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

Side by Side Diff: Source/modules/mediastream/RTCPeerConnection.h

Issue 1149293003: Oilpan: eagerly sweep some mediasource and mediastream objects. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Updated to use EAGERLY_FINALIZE() 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2012 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 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. 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 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 127 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 // ActiveDOMObject 138 // ActiveDOMObject
139 virtual void suspend() override; 139 virtual void suspend() override;
140 virtual void resume() override; 140 virtual void resume() override;
141 virtual void stop() override; 141 virtual void stop() override;
142 // We keep the this object alive until either stopped or closed. 142 // We keep the this object alive until either stopped or closed.
143 virtual bool hasPendingActivity() const override 143 virtual bool hasPendingActivity() const override
144 { 144 {
145 return !m_closed && !m_stopped; 145 return !m_closed && !m_stopped;
146 } 146 }
147 147
148 // Oilpan: need to eagerly finalize m_peerHandler
149 EAGERLY_FINALIZE();
148 DECLARE_VIRTUAL_TRACE(); 150 DECLARE_VIRTUAL_TRACE();
149 151
150 private: 152 private:
151 RTCPeerConnection(ExecutionContext*, RTCConfiguration*, WebMediaConstraints, ExceptionState&); 153 RTCPeerConnection(ExecutionContext*, RTCConfiguration*, WebMediaConstraints, ExceptionState&);
152 154
153
154 static RTCConfiguration* parseConfiguration(const Dictionary&, ExceptionStat e&); 155 static RTCConfiguration* parseConfiguration(const Dictionary&, ExceptionStat e&);
155 static RTCOfferOptions* parseOfferOptions(const Dictionary&, ExceptionState& ); 156 static RTCOfferOptions* parseOfferOptions(const Dictionary&, ExceptionState& );
156 157
157 void scheduleDispatchEvent(PassRefPtrWillBeRawPtr<Event>); 158 void scheduleDispatchEvent(PassRefPtrWillBeRawPtr<Event>);
158 void dispatchScheduledEvent(); 159 void dispatchScheduledEvent();
159 bool hasLocalStreamWithTrackId(const String& trackId); 160 bool hasLocalStreamWithTrackId(const String& trackId);
160 161
161 void changeSignalingState(WebRTCPeerConnectionHandlerClient::SignalingState) ; 162 void changeSignalingState(WebRTCPeerConnectionHandlerClient::SignalingState) ;
162 void changeIceGatheringState(WebRTCPeerConnectionHandlerClient::ICEGathering State); 163 void changeIceGatheringState(WebRTCPeerConnectionHandlerClient::ICEGathering State);
163 void changeIceConnectionState(WebRTCPeerConnectionHandlerClient::ICEConnecti onState); 164 void changeIceConnectionState(WebRTCPeerConnectionHandlerClient::ICEConnecti onState);
(...skipping 14 matching lines...) Expand all
178 AsyncMethodRunner<RTCPeerConnection> m_dispatchScheduledEventRunner; 179 AsyncMethodRunner<RTCPeerConnection> m_dispatchScheduledEventRunner;
179 WillBeHeapVector<RefPtrWillBeMember<Event>> m_scheduledEvents; 180 WillBeHeapVector<RefPtrWillBeMember<Event>> m_scheduledEvents;
180 181
181 bool m_stopped; 182 bool m_stopped;
182 bool m_closed; 183 bool m_closed;
183 }; 184 };
184 185
185 } // namespace blink 186 } // namespace blink
186 187
187 #endif // RTCPeerConnection_h 188 #endif // RTCPeerConnection_h
OLDNEW
« Source/modules/mediastream/MediaStreamTrack.h ('K') | « Source/modules/mediastream/RTCDataChannel.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698