Move settings-section animations into setting-section, make better

chrome/browser/resources/settings/settings_page/transition_behavior.js

Index: chrome/browser/resources/settings/settings_page/transition_behavior.js
diff --git a/chrome/browser/resources/settings/settings_page/transition_behavior.js b/chrome/browser/resources/settings/settings_page/transition_behavior.js
deleted file mode 100644
index 0aecfb3968ee342a04163e911c57f29de67c9901..0000000000000000000000000000000000000000
--- a/chrome/browser/resources/settings/settings_page/transition_behavior.js
+++ /dev/null
@@ -1,86 +0,0 @@
-// 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.
-
-/**
- * Adds convenience functions to control native Web Animations. The
- * implementation details may change as Chrome support evolves.
- * @polymerBehavior
- */
-var TransitionBehavior = {
-  ready: function() {
-    /**
-     * @type {!Object<!Animation>}
-     * Map of running animations by animation name. Animation names are
-     * arbitrary but used to prevent multiple animations of the same type (e.g.,
-     * expand/collapse section) from running simultaneously.
-     */
-    this.animations = {};
-
-    /**
-     * @private {!Object<!Promise>}
-     * Map of Promises for each running animation. Key names and existence
-     * should correspond with |animations|.
-     */
-    this.promises_ = {};
-  },
-
-  /**
-   * Calls el.animate(keyframes, opt_options). Returns a Promise which is
-   * resolved when the transition ends, or rejected when the transition is
-   * canceled. If an animation with the same name is in progress, that
-   * animation is finished immediately before creating the new animation.
-   * @param {string} name Name of the animation, used to finish this animation
-   *     when playing the same type of animation again.
-   * @param {!HTMLElement} el The element to animate.
-   * @param {!Array|!Object} keyframes Keyframes, as in Element.animate.
-   * @param {number|!KeyframeEffectOptions=} opt_options Options, as in
-   *     Element.animate.
-   * @return {!Promise} A promise which is resolved when the animation finishes
-   *     or rejected if the animation is canceled.
-   */
-  animateElement: function(name, el, keyframes, opt_options) {
-    if (this.animations[name])
-      this.animations[name].finish();
-
-    var animation = el.animate(keyframes, opt_options);
-    this.animations[name] = animation;
-
-    this.promises_[name] = new Promise(function(resolve, reject) {
-      animation.addEventListener('finish', function() {
-        this.animations[name] = undefined;
-        resolve();
-      }.bind(this));
-
-      animation.addEventListener('cancel', function() {
-        this.animations[name] = undefined;
-        reject();

[Dan Beam, 2016/07/07 18:27:54]

we probably shouldn't be rejecting for this.  i think we should just be
resolving with an argument of whether the animation succeeded

[Dan Beam, 2016/07/12 01:36:48]

ping

[michaelpg, 2016/07/20 22:10:57]

?? this whole file was deleted. you had a similar comment about
c/b/r/settings/animation/animation.js, but I pointed out that's the behavior
prescribed by the Animation interface.

-      }.bind(this));
-    }.bind(this));
-    return this.promises_[name];
-  },
-
-  /**
-   * Finishes the ongoing named animation, waits for the animation's Promise
-   * to be resolved, then calls the callback.
-   * @param {string} name Name of the animation passed to animateElement.
-   * @param {function()=} opt_callback Called once the animation finishes.
-   */
-  finishAnimation: function(name, opt_callback) {
-    if (opt_callback)
-      this.promises_[name].then(opt_callback);
-    this.animations[name].finish();
-  },
-
-  /**
-   * Cancels the ongoing named animation, waits for the animation's Promise
-   * to be rejected, then calls the callback.
-   * @param {string} name Name of the animation passed to animateElement.
-   * @param {function()=} opt_callback Called once the animation cancels.
-   */
-  cancelAnimation: function(name, opt_callback) {
-    if (opt_callback)
-      this.promises_[name].catch(opt_callback);
-    this.animations[name].cancel();
-  },
-};