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

Unified Diff: Source/modules/mediastream/MediaStreamTrack.cpp

Issue 16778002: MediaStream API: Changing the device enumeration to be async (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Review comment fixed Created 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/modules/mediastream/MediaStreamTrack.h ('k') | Source/modules/mediastream/MediaStreamTrack.idl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/mediastream/MediaStreamTrack.cpp
diff --git a/Source/modules/mediastream/MediaStreamTrack.cpp b/Source/modules/mediastream/MediaStreamTrack.cpp
index fe6ceafc8095ff6050c78c78de8e9f6de86cda92..a0aaca3371dcb326be670da494de3a5d483e6a2f 100644
--- a/Source/modules/mediastream/MediaStreamTrack.cpp
+++ b/Source/modules/mediastream/MediaStreamTrack.cpp
@@ -30,8 +30,9 @@
#include "core/dom/ScriptExecutionContext.h"
#include "core/platform/mediastream/MediaStreamCenter.h"
#include "core/platform/mediastream/MediaStreamComponent.h"
+#include "modules/mediastream/MediaStreamTrackSourcesCallback.h"
+#include "modules/mediastream/MediaStreamTrackSourcesRequest.h"
#include "public/platform/WebSourceInfo.h"
-#include "weborigin/SecurityOrigin.h"
namespace WebCore {
@@ -118,20 +119,12 @@ String MediaStreamTrack::readyState() const
return String();
}
-SourceInfoVector MediaStreamTrack::getSourceInfos(ScriptExecutionContext* context, ExceptionCode& ec)
+void MediaStreamTrack::getSources(ScriptExecutionContext* context, PassRefPtr<MediaStreamTrackSourcesCallback> callback, ExceptionCode& ec)
{
- WebKit::WebVector<WebKit::WebSourceInfo> webSourceInfos;
- bool ok = MediaStreamCenter::instance().getSourceInfos(context->securityOrigin()->toString(), webSourceInfos);
- if (!ok) {
+ RefPtr<MediaStreamTrackSourcesRequest> request = MediaStreamTrackSourcesRequest::create(context, callback);
+ bool ok = MediaStreamCenter::instance().getMediaStreamTrackSources(request.release());
+ if (!ok)
ec = NOT_SUPPORTED_ERR;
- return SourceInfoVector();
- }
-
- SourceInfoVector sourceInfos;
- for (size_t i = 0; i < webSourceInfos.size(); ++i)
- sourceInfos.append(SourceInfo::create(webSourceInfos[i]));
-
- return sourceInfos;
}
bool MediaStreamTrack::ended() const
« no previous file with comments | « Source/modules/mediastream/MediaStreamTrack.h ('k') | Source/modules/mediastream/MediaStreamTrack.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698