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

Unified Diff: third_party/WebKit/public/platform/WebAudioLatencyHint.h

Issue 2501863003: Support for AudioContextOptions latencyHint. (Closed)
Patch Set: Updates to Web Audio code. Created 4 years 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 | « third_party/WebKit/public/platform/WebAudioDevice.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/public/platform/WebAudioLatencyHint.h
diff --git a/third_party/WebKit/public/platform/WebAudioLatencyHint.h b/third_party/WebKit/public/platform/WebAudioLatencyHint.h
new file mode 100644
index 0000000000000000000000000000000000000000..5bc386daf6548806f65a67c02743ce12e298ffa6
--- /dev/null
+++ b/third_party/WebKit/public/platform/WebAudioLatencyHint.h
@@ -0,0 +1,49 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef WebAudioLatencyHint_h
+#define WebAudioLatencyHint_h
+
+#include "WebString.h"
+
+namespace blink {
+
+class WebAudioLatencyHint {
+ public:
+ enum Category {
Raymond Toy 2016/12/13 15:53:11 Category seems a bit terse. Maybe AudioLatencyCat
Andrew MacPherson 2016/12/14 09:07:20 Done, with AudioContextLatencyCategory.
+ CategoryInteractive,
+ CategoryBalanced,
+ CategoryPlayback,
+ CategoryExact
+ };
Raymond Toy 2016/12/13 15:53:11 https://www.chromium.org/blink/coding-style, item
Andrew MacPherson 2016/12/14 09:07:20 Done.
+
+ explicit WebAudioLatencyHint(WebString category) {
+ if (category == "interactive") {
+ m_category = CategoryInteractive;
+ } else if (category == "balanced") {
+ m_category = CategoryBalanced;
+ } else if (category == "playback") {
+ m_category = CategoryPlayback;
+ } else {
Raymond Toy 2016/12/13 15:53:11 I think you need a case for category == CategoryEx
Andrew MacPherson 2016/12/14 09:07:20 Should it be possible to create a WebAudioLatencyH
Raymond Toy 2016/12/14 18:23:32 Sorry, you're right. It shouldn't be possible, and
+ NOTREACHED();
+ m_category = CategoryInteractive;
+ }
+ }
+
+ explicit WebAudioLatencyHint(Category category)
+ : m_category(category), m_seconds(0) {}
+ explicit WebAudioLatencyHint(double seconds)
+ : m_category(CategoryExact), m_seconds(seconds) {}
+
+ Category category() const { return m_category; }
+ double seconds() const { return m_seconds; }
Raymond Toy 2016/12/13 15:53:11 Should there be asserts here so that you only ask
Andrew MacPherson 2016/12/14 09:07:20 Done.
+
+ private:
+ Category m_category;
+ double m_seconds;
+};
+
+} // namespace blink
+
+#endif
« no previous file with comments | « third_party/WebKit/public/platform/WebAudioDevice.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698