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

Unified Diff: third_party/WebKit/Source/platform/mediastream/MediaStreamComponent.cpp

Issue 1363143003: [Oilpan] Move MediaStream{Source|Component|Descriptor} to Oilpan heap (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Revert changes in PS8 Created 5 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/mediastream/MediaStreamComponent.cpp
diff --git a/third_party/WebKit/Source/platform/mediastream/MediaStreamComponent.cpp b/third_party/WebKit/Source/platform/mediastream/MediaStreamComponent.cpp
index 8e6721cb71c1418808110181eee19544f0b79706..b6d59cdf37a07a14bbc42a2b56bbeb2fa8c419ab 100644
--- a/third_party/WebKit/Source/platform/mediastream/MediaStreamComponent.cpp
+++ b/third_party/WebKit/Source/platform/mediastream/MediaStreamComponent.cpp
@@ -39,17 +39,17 @@
namespace blink {
-PassRefPtr<MediaStreamComponent> MediaStreamComponent::create(PassRefPtr<MediaStreamSource> source)
+MediaStreamComponent* MediaStreamComponent::create(MediaStreamSource* source)
{
- return adoptRef(new MediaStreamComponent(createCanonicalUUIDString(), source));
+ return new MediaStreamComponent(createCanonicalUUIDString(), source);
}
-PassRefPtr<MediaStreamComponent> MediaStreamComponent::create(const String& id, PassRefPtr<MediaStreamSource> source)
+MediaStreamComponent* MediaStreamComponent::create(const String& id, MediaStreamSource* source)
{
- return adoptRef(new MediaStreamComponent(id, source));
+ return new MediaStreamComponent(id, source);
}
-MediaStreamComponent::MediaStreamComponent(const String& id, PassRefPtr<MediaStreamSource> source)
+MediaStreamComponent::MediaStreamComponent(const String& id, MediaStreamSource* source)
: m_source(source)
, m_id(id)
, m_enabled(true)
@@ -87,5 +87,9 @@ void MediaStreamComponent::AudioSourceProviderImpl::provideInput(AudioBus* bus,
}
#endif // #if ENABLE(WEB_AUDIO)
-} // namespace blink
+DEFINE_TRACE(MediaStreamComponent)
+{
+ visitor->trace(m_source);
+}
+} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698