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

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

Issue 2050123002: Remove OwnPtr from Blink. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: First attempt to land. 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) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 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 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 12 matching lines...) Expand all
23 */ 23 */
24 24
25 #include "public/platform/WebMediaStreamTrack.h" 25 #include "public/platform/WebMediaStreamTrack.h"
26 26
27 #include "platform/mediastream/MediaStreamComponent.h" 27 #include "platform/mediastream/MediaStreamComponent.h"
28 #include "platform/mediastream/MediaStreamSource.h" 28 #include "platform/mediastream/MediaStreamSource.h"
29 #include "public/platform/WebAudioSourceProvider.h" 29 #include "public/platform/WebAudioSourceProvider.h"
30 #include "public/platform/WebMediaStream.h" 30 #include "public/platform/WebMediaStream.h"
31 #include "public/platform/WebMediaStreamSource.h" 31 #include "public/platform/WebMediaStreamSource.h"
32 #include "public/platform/WebString.h" 32 #include "public/platform/WebString.h"
33 #include "wtf/PtrUtil.h"
34 #include <memory>
33 35
34 namespace blink { 36 namespace blink {
35 37
36 namespace { 38 namespace {
37 39
38 class TrackDataContainer : public MediaStreamComponent::TrackData { 40 class TrackDataContainer : public MediaStreamComponent::TrackData {
39 public: 41 public:
40 explicit TrackDataContainer(PassOwnPtr<WebMediaStreamTrack::TrackData> e xtraData) 42 explicit TrackDataContainer(std::unique_ptr<WebMediaStreamTrack::TrackDa ta> extraData)
41 : m_extraData(std::move(extraData)) 43 : m_extraData(std::move(extraData))
42 { 44 {
43 } 45 }
44 46
45 WebMediaStreamTrack::TrackData* getTrackData() { return m_extraData.get( ); } 47 WebMediaStreamTrack::TrackData* getTrackData() { return m_extraData.get( ); }
46 void getSettings(WebMediaStreamTrack::Settings& settings) 48 void getSettings(WebMediaStreamTrack::Settings& settings)
47 { 49 {
48 m_extraData->getSettings(settings); 50 m_extraData->getSettings(settings);
49 } 51 }
50 52
51 private: 53 private:
52 OwnPtr<WebMediaStreamTrack::TrackData> m_extraData; 54 std::unique_ptr<WebMediaStreamTrack::TrackData> m_extraData;
53 }; 55 };
54 56
55 } // namespace 57 } // namespace
56 58
57 WebMediaStreamTrack::WebMediaStreamTrack(MediaStreamComponent* mediaStreamCompon ent) 59 WebMediaStreamTrack::WebMediaStreamTrack(MediaStreamComponent* mediaStreamCompon ent)
58 : m_private(mediaStreamComponent) 60 : m_private(mediaStreamComponent)
59 { 61 {
60 } 62 }
61 63
62 WebMediaStreamTrack& WebMediaStreamTrack::operator=(MediaStreamComponent* mediaS treamComponent) 64 WebMediaStreamTrack& WebMediaStreamTrack::operator=(MediaStreamComponent* mediaS treamComponent)
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 MediaStreamComponent::TrackData* data = m_private->getTrackData(); 116 MediaStreamComponent::TrackData* data = m_private->getTrackData();
115 if (!data) 117 if (!data)
116 return 0; 118 return 0;
117 return static_cast<TrackDataContainer*>(data)->getTrackData(); 119 return static_cast<TrackDataContainer*>(data)->getTrackData();
118 } 120 }
119 121
120 void WebMediaStreamTrack::setTrackData(TrackData* extraData) 122 void WebMediaStreamTrack::setTrackData(TrackData* extraData)
121 { 123 {
122 ASSERT(!m_private.isNull()); 124 ASSERT(!m_private.isNull());
123 125
124 m_private->setTrackData(adoptPtr(new TrackDataContainer(adoptPtr(extraData)) )); 126 m_private->setTrackData(wrapUnique(new TrackDataContainer(wrapUnique(extraDa ta))));
125 } 127 }
126 128
127 void WebMediaStreamTrack::setSourceProvider(WebAudioSourceProvider* provider) 129 void WebMediaStreamTrack::setSourceProvider(WebAudioSourceProvider* provider)
128 { ASSERT(!m_private.isNull()); 130 { ASSERT(!m_private.isNull());
129 m_private->setSourceProvider(provider); 131 m_private->setSourceProvider(provider);
130 } 132 }
131 133
132 void WebMediaStreamTrack::assign(const WebMediaStreamTrack& other) 134 void WebMediaStreamTrack::assign(const WebMediaStreamTrack& other)
133 { 135 {
134 m_private = other.m_private; 136 m_private = other.m_private;
135 } 137 }
136 138
137 } // namespace blink 139 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698