[cros] Layout for OOBE WebUI.

chrome/browser/resources/chromeos/oobe.js

+// Copyright (c) 2011 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.
+
+/**
+ * @fileoverview Out of the box experience flow (OOBE).
+ * This is the main code for the OOBE WebUI implementation.
+ */
+
+const steps = ['Connect', 'Eula', 'Update'];
+
+cr.define('cr.ui', function() {
+  cr.addSingletonGetter(Oobe);
+
+  Oobe.localStrings_ = new LocalStrings();
+
+  Oobe.prototype = {
+    /**
+     * Current OOBE step, index in the steps array.
+     * @type {number}
+     */
+    currentStep_: 0,
+
+    /**
+     * Switches to the next OOBE step.
+     * @param {number} nextStep Index of the next step.
+     */
+    toggleStep_: function(nextStep) {
+      var offset = nextStep - this.currentStep_;
+      var oldstep = $(steps[this.currentStep_]);
+      var oldheader = $('h' + steps[this.currentStep_]);
+      var newstep = $(steps[this.currentStep_ + offset]);
+      var newheader = $('h' + steps[this.currentStep_ + offset]);
+
+      newstep.classList.remove('hidden');
+
+      // TODO(nkostylev): Support offset other than -1/1.
+      if (offset == 1) {
+        oldheader.classList.add('left');
+        oldstep.classList.add('left');
+        newheader.classList.remove('right');
+        newstep.classList.remove('right');
+      } else if (offset == -1) {
+        oldheader.classList.add('right');
+        oldstep.classList.add('right');
+        newheader.classList.remove('left');
+        newstep.classList.remove('left');
+      }
+
+      // Adjust inner container height based on new step's height.
+      $('inner-container').style.height =
+          $(steps[this.currentStep_ + offset]).offsetHeight;
+
+      setTimeout(function(){oldstep.classList.add('hidden');}, 500);

[Evan Stade, 2011/06/08 22:46:56]

this is not correct. I don't even think it's necessary, is it?

If you do have a need for it, then please connect to the webkit transition end
event instead of setting a timeout.

[Nikita (slow), 2011/06/09 15:52:14]

Done.

+      this.currentStep_ += offset;

[Evan Stade, 2011/06/08 22:46:56]

I guess this is a pretty insignificant nit, but it seems kinda roundabout to
apply the offset instead of just setting = nextStep. The non-intuitiveness made
me go back and check if there was some other magic happening with the value of
offset.

[Nikita (slow), 2011/06/09 15:52:14]

You're right, I'll change this. Initially offset was passed as a parameter, but
I forgot to update it here, Done.

+      $('oobe').className = steps[this.currentStep_];
+    },
+  };
+
+  /**
+   * Initializes the OOBE flow.  This will cause all C++ handlers to
+   * be invoked to do final setup.
+   */
+  Oobe.initialize = function() {
+    // Adjust inner container height based on first step's height
+    $('inner-container').style.height = $(steps[0]).offsetHeight;
+
+    $('continue-button').addEventListener('click', function(event) {
+      // TODO(nkostylev): Callback screen handler.
+      Oobe.toggleStep(1);
+    });
+    $('back-button').addEventListener('click', function(event) {
+      // TODO(nkostylev): Callback screen handler.
+      Oobe.toggleStep(0);
+    });
+    $('accept-button').addEventListener('click', function(event) {
+      // TODO(nkostylev): Callback screen handler.
+      Oobe.toggleStep(2);
+    });
+
+    chrome.send('screenStateInitialize');
+  };
+
+  /**
+   * Switches to the next OOBE step.
+   * @param {number} nextStep Index of the next step.
+   */
+  Oobe.toggleStep = function(nextStep) {
+    Oobe.getInstance().toggleStep_(nextStep);
+  };
+
+  // Export
+  return {
+    Oobe: Oobe
+  };
+});
+
+var Oobe = cr.ui.Oobe;
+
+document.addEventListener('DOMContentLoaded', cr.ui.Oobe.initialize);