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

Unified Diff: chrome/browser/resources/hotword_audio_verification/flow.js

Issue 521483003: Hotword Audio Verification App: Adds hotwording assets and basic opt-in flow. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: No copies for image files Created 6 years, 4 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: chrome/browser/resources/hotword_audio_verification/flow.js
diff --git a/chrome/browser/resources/hotword_audio_verification/flow.js b/chrome/browser/resources/hotword_audio_verification/flow.js
new file mode 100644
index 0000000000000000000000000000000000000000..09fc1b15e5d1101954eec2f51caac8002b685909
--- /dev/null
+++ b/chrome/browser/resources/hotword_audio_verification/flow.js
@@ -0,0 +1,87 @@
+// Copyright 2014 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.
+
+(function(context) {
+
+ // Correspond to steps in the hotword opt-in flow.
+ /** @const */ var HOTWORD_AUDIO_HISTORY = 'hotword-audio-history-container';
+ /** @const */ var HOTWORD_ONLY_START = 'hotword-only-container';
+ /** @const */ var AUDIO_HISTORY_START = 'audio-history-container';
+ /** @const */ var SPEECH_TRAINING = 'speech-training-container';
+ /** @const */ var FINISHED = 'finished-container';
+
+ /** @const */ var FLOWS = {
+ HOTWORD_AND_AUDIO_HISTORY: [
+ HOTWORD_AUDIO_HISTORY, SPEECH_TRAINING, FINISHED],
+ HOTWORD_ONLY: [HOTWORD_ONLY_START, SPEECH_TRAINING, FINISHED],
+ AUDIO_HISTORY_ONLY: [AUDIO_HISTORY_START]
+ };
+
+ /**
+ * Class to control the page flow of the always-on hotword and
+ * Audio History opt-in process.
+ * @constructor
+ */
+ function Flow() {
+ this.currentStepIndex_ = -1;
+ this.currentFlow_ = [];
+ }
+
+ /**
+ * Gets the appropriate flow and displays its first page.
+ */
+ Flow.prototype.startFlow = function() {
+ this.currentFlow_ = getFlowForSetting_.apply(this);
+ this.advanceStep();
+ };
+
+ /**
+ * Advances the current step.
+ * @return {boolean} true if we have not reached the end of the flow.
Bernhard Bauer 2014/09/01 10:16:23 What is the return value used for?
kcarattini 2014/09/02 04:37:23 Removed.
+ */
+ Flow.prototype.advanceStep = function() {
+ if (this.currentStepIndex_ >= 0) {
+ this.currentStepIndex_++;
+ if (this.currentStepIndex_ > this.currentFlow_.length) {
Bernhard Bauer 2014/09/01 10:16:23 No braces around single-line bodies.
kcarattini 2014/09/02 04:37:23 Done.
+ return false;
+ }
+ } else {
+ this.currentStepIndex_ = 0;
Bernhard Bauer 2014/09/01 10:16:23 Are there other possible negative values than -1?
kcarattini 2014/09/02 04:37:23 Done.
+ }
+
+ showStep_.apply(this);
+ return true;
+ };
+
+ // ---- private methods:
+
+ /**
+ * Gets the appropriate flow for the current configuration of settings.
+ * @private
+ */
+ getFlowForSetting_ = function() {
+ // TODO(kcarattini): This should eventually return the correct flow for
+ // the current settings state.
+ return FLOWS.HOTWORD_AND_AUDIO_HISTORY;
+ };
+
+ /**
+ * Displays the current step. If the current step is not the first step.
Bernhard Bauer 2014/09/01 10:16:23 I think there should be a comma at the end here.
kcarattini 2014/09/02 04:37:23 Done.
+ * also hides the previous step.
+ * @private
+ */
+ showStep_ = function() {
+ var currentStep = this.currentFlow_[this.currentStepIndex_];
+ var previousStep = null;
+ if (this.currentStepIndex_ > 0)
+ previousStep = this.currentFlow_[this.currentStepIndex_ - 1];
+
+ if (previousStep)
+ document.getElementById(previousStep).hidden = true;
+
+ document.getElementById(currentStep).hidden = false;
+ };
+
+ window.Flow = Flow;
Bernhard Bauer 2014/09/01 10:16:23 Do you want to use |context| here? Or just leave t
kcarattini 2014/09/02 04:37:23 Done.
+})(window);

Powered by Google App Engine
This is Rietveld 408576698