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

Side by Side Diff: third_party/WebKit/Source/platform/exported/WebMediaStreamSource.cpp

Issue 2080623002: Revert "Remove OwnPtr from Blink." (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
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 17 matching lines...) Expand all
28 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 28 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
29 */ 29 */
30 30
31 #include "public/platform/WebMediaStreamSource.h" 31 #include "public/platform/WebMediaStreamSource.h"
32 32
33 #include "platform/audio/AudioBus.h" 33 #include "platform/audio/AudioBus.h"
34 #include "platform/mediastream/MediaStreamSource.h" 34 #include "platform/mediastream/MediaStreamSource.h"
35 #include "public/platform/WebAudioDestinationConsumer.h" 35 #include "public/platform/WebAudioDestinationConsumer.h"
36 #include "public/platform/WebMediaConstraints.h" 36 #include "public/platform/WebMediaConstraints.h"
37 #include "public/platform/WebString.h" 37 #include "public/platform/WebString.h"
38 #include "wtf/PtrUtil.h" 38 #include "wtf/PassOwnPtr.h"
39 #include "wtf/Vector.h" 39 #include "wtf/Vector.h"
40 #include <memory>
41 40
42 namespace blink { 41 namespace blink {
43 42
44 namespace { 43 namespace {
45 44
46 class ExtraDataContainer : public MediaStreamSource::ExtraData { 45 class ExtraDataContainer : public MediaStreamSource::ExtraData {
47 public: 46 public:
48 ExtraDataContainer(std::unique_ptr<WebMediaStreamSource::ExtraData> extraDat a) : m_extraData(std::move(extraData)) { } 47 ExtraDataContainer(PassOwnPtr<WebMediaStreamSource::ExtraData> extraData) : m_extraData(std::move(extraData)) { }
49 48
50 WebMediaStreamSource::ExtraData* getExtraData() { return m_extraData.get(); } 49 WebMediaStreamSource::ExtraData* getExtraData() { return m_extraData.get(); }
51 50
52 private: 51 private:
53 std::unique_ptr<WebMediaStreamSource::ExtraData> m_extraData; 52 OwnPtr<WebMediaStreamSource::ExtraData> m_extraData;
54 }; 53 };
55 54
56 } // namespace 55 } // namespace
57 56
58 WebMediaStreamSource WebMediaStreamSource::ExtraData::owner() 57 WebMediaStreamSource WebMediaStreamSource::ExtraData::owner()
59 { 58 {
60 ASSERT(m_owner); 59 ASSERT(m_owner);
61 return WebMediaStreamSource(m_owner); 60 return WebMediaStreamSource(m_owner);
62 } 61 }
63 62
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 return static_cast<ExtraDataContainer*>(data)->getExtraData(); 147 return static_cast<ExtraDataContainer*>(data)->getExtraData();
149 } 148 }
150 149
151 void WebMediaStreamSource::setExtraData(ExtraData* extraData) 150 void WebMediaStreamSource::setExtraData(ExtraData* extraData)
152 { 151 {
153 ASSERT(!m_private.isNull()); 152 ASSERT(!m_private.isNull());
154 153
155 if (extraData) 154 if (extraData)
156 extraData->setOwner(m_private.get()); 155 extraData->setOwner(m_private.get());
157 156
158 m_private->setExtraData(wrapUnique(new ExtraDataContainer(wrapUnique(extraDa ta)))); 157 m_private->setExtraData(adoptPtr(new ExtraDataContainer(adoptPtr(extraData)) ));
159 } 158 }
160 159
161 WebMediaConstraints WebMediaStreamSource::constraints() 160 WebMediaConstraints WebMediaStreamSource::constraints()
162 { 161 {
163 ASSERT(!m_private.isNull()); 162 ASSERT(!m_private.isNull());
164 return m_private->constraints(); 163 return m_private->constraints();
165 } 164 }
166 165
167 bool WebMediaStreamSource::requiresAudioConsumer() const 166 bool WebMediaStreamSource::requiresAudioConsumer() const
168 { 167 {
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
226 ConsumerWrapper* wrapper = static_cast<ConsumerWrapper*>(it->get()); 225 ConsumerWrapper* wrapper = static_cast<ConsumerWrapper*>(it->get());
227 if (wrapper->consumer() == consumer) { 226 if (wrapper->consumer() == consumer) {
228 m_private->removeAudioConsumer(wrapper); 227 m_private->removeAudioConsumer(wrapper);
229 return true; 228 return true;
230 } 229 }
231 } 230 }
232 return false; 231 return false;
233 } 232 }
234 233
235 } // namespace blink 234 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698