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

Side by Side Diff: third_party/WebKit/public/platform/WebMediaStreamTrack.h

Issue 2606983002: Media Capture Depth Stream Extensions API: focal length and depth range. (Closed)
Patch Set: review #25 fix. Thanks kinuko@. Created 3 years, 11 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 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 struct Settings { 44 struct Settings {
45 bool hasFrameRate() { return frameRate >= 0.0; } 45 bool hasFrameRate() { return frameRate >= 0.0; }
46 bool hasWidth() { return width >= 0; } 46 bool hasWidth() { return width >= 0; }
47 bool hasHeight() { return height >= 0; } 47 bool hasHeight() { return height >= 0; }
48 // The variables are read from 48 // The variables are read from
49 // MediaStreamTrack::GetSettings only. 49 // MediaStreamTrack::GetSettings only.
50 double frameRate = -1.0; 50 double frameRate = -1.0;
51 long width = -1; 51 long width = -1;
52 long height = -1; 52 long height = -1;
53 WebString deviceId; 53 WebString deviceId;
54 // Media Capture Depth Stream Extensions.
55 double focalLengthX = 0.0;
56 double focalLengthY = 0.0;
57 double depthNear = 0.0;
58 double depthFar = 0.0;
mcasas 2017/01/11 23:18:20 What happened to |focalLength|? https://www.w3.or
aleksandar.stojiljkovic 2017/01/12 14:13:59 Replaced [focalLength, horizontalFieldOfView and v
54 }; 59 };
55 60
56 class TrackData { 61 class TrackData {
57 public: 62 public:
58 TrackData() {} 63 TrackData() {}
59 virtual ~TrackData() {} 64 virtual ~TrackData() {}
60 virtual void getSettings(Settings&) = 0; 65 virtual void getSettings(Settings&) = 0;
61 }; 66 };
62 67
63 enum class ContentHintType { 68 enum class ContentHintType {
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
111 BLINK_PLATFORM_EXPORT operator MediaStreamComponent*() const; 116 BLINK_PLATFORM_EXPORT operator MediaStreamComponent*() const;
112 #endif 117 #endif
113 118
114 private: 119 private:
115 WebPrivatePtr<MediaStreamComponent> m_private; 120 WebPrivatePtr<MediaStreamComponent> m_private;
116 }; 121 };
117 122
118 } // namespace blink 123 } // namespace blink
119 124
120 #endif // WebMediaStreamTrack_h 125 #endif // WebMediaStreamTrack_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698