Hotword Audio Verification App: Adds hotwording assets and basic opt-in flow.

chrome/browser/resources/hotword_audio_verification/flow.js

+// 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? If not, this branch isn't
necessary (the if branch above will also set the value to 0, and as it's not
possible to have a negative length, won't return false).

[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 the parameter out.

[kcarattini, 2014/09/02 04:37:23]

Done.

+})(window);