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

Unified Diff: webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMTLVideoView.h

Issue 2722583002: Add Metal video view in AppRTCMobile and metal availability macro. (Closed)
Patch Set: Created 3 years, 10 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: webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMTLVideoView.h
diff --git a/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMTLVideoView.h b/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMTLVideoView.h
index 681c457516fdf040770904207659cb55be46ba0d..17f307452d70339ff8d5dfb6ddb2cfcd89660bb2 100644
--- a/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMTLVideoView.h
+++ b/webrtc/sdk/objc/Framework/Headers/WebRTC/RTCMTLVideoView.h
@@ -12,9 +12,18 @@
#import "WebRTC/RTCVideoRenderer.h"
-NS_ASSUME_NONNULL_BEGIN
RTC_EXPORT
kthelgason 2017/02/28 10:03:10 This should be right before the exported symbol, i
+// Check if metal is supported in WebRTC.
+// NOTE: Currently __arm64__ == Metal.
+#if defined(__arm64__) && __arm64__
+ #define RTC_SUPPORTS_METAL 1
+#else
+ #define RTC_SUPPORTS_METAL 0
+#endif
kthelgason 2017/02/28 10:03:10 I'd want to set this in gn if possible. Are there
daniela-webrtc 2017/02/28 14:27:12 This is intended for the consumers of the API, to
+
+NS_ASSUME_NONNULL_BEGIN
+
/**
* RTCMTLVideoView is thin wrapper around MTKView.
*

Powered by Google App Engine
This is Rietveld 408576698