Merge pull request #124 from alancutter/revertUseStrict

Revert adding 'use strict'

Author: alancutter

History.md

@@ -1,3 +1,7 @@
+### 2.2.4 - *April 14 2017*
+
+  *  [Reverted adding 'use strict' to source files.](https://github.com/web-animations/web-animations-next/pull/124)
+
 ### 2.2.3 - *April 13 2017*
 
   *  [Added HTML import targets.](https://github.com/web-animations/web-animations-js/pull/94)

package.json

@@ -5,7 +5,7 @@
     "type": "git",
     "url": "https://github.com/web-animations/web-animations-js.git"
   },
-  "version": "2.2.3",
+  "version": "2.2.4",
   "keywords": [
     "animations",
     "polyfill"

src/animation.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(shared, scope, testing) {
 
   shared.sequenceNumber = 0;
@@ -192,6 +191,7 @@
       this._inEffect = false;
       this._idle = true;
       this._paused = false;
+      this._isFinished = true;
       this._finishedFlag = true;
       this._currentTime = 0;
       this._startTime = null;

src/apply-preserving-inline-style.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
 
   var styleAttributes = {

src/apply.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
 
   scope.apply = function(element, property, value) {

src/box-handler.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
   function consumeLengthPercentOrAuto(string) {
     return scope.consumeLengthOrPercent(string) || scope.consumeToken(/^auto/, string);

src/color-handler.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
 
   var canvas = document.createElementNS('http://www.w3.org/1999/xhtml', 'canvas');

src/deprecation.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(shared) {
 
   var silenced = {};

src/dev.js

@@ -12,6 +12,5 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 var WEB_ANIMATIONS_TESTING = false;
 var webAnimationsTesting = null;

src/dimension-handler.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
 
   function parseDimension(unitRegExp, string) {

src/effect-callback.js

@@ -11,8 +11,6 @@
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 //     See the License for the specific language governing permissions and
 // limitations under the License.
-
-'use strict';
 (function(shared, scope, testing) {
 
   var nullTarget = document.createElementNS('http://www.w3.org/1999/xhtml', 'div');

src/element-animatable.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope) {
   window.Element.prototype.animate = function(effectInput, options) {
     var id = '';

src/font-weight-handler.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope) {
   function parse(string) {
     var out = Number(string);

src/group-constructors.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(shared, scope, testing) {
 
   function groupChildDuration(node) {
@@ -117,8 +116,7 @@
     },
     remove: function() {
       scope.removeMulti([this]);
-    },
-    _updateActiveDuration: function() {}
+    }
   };
 
   window.SequenceEffect.prototype = Object.create(constructor.prototype);

src/handler-utils.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope) {
 
   // consume* functions return a 2 value array of [parsed-data, '' or not-yet consumed input]

src/interpolation.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
 
   function interpolate(from, to, f) {

src/keyframe-effect-constructor.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(shared, scope, testing) {
 
   var disassociate = function(effect) {
@@ -71,7 +70,7 @@
       this._normalizedKeyframes = new KeyframeList(effectInput);
     }
     this._keyframes = effectInput;
-    this._updateActiveDuration();
+    this.activeDuration = shared.calculateActiveDuration(this._timing);
     this._id = id;
     return this;
   };
@@ -94,9 +93,6 @@
     get parent() {
       return this._parent;
     },
-    get activeDuration() {
-      return this._activeDuration;
-    },
     clone: function() {
       if (typeof this.getFrames() == 'function') {
         throw new Error('Cloning custom effects is not supported.');
@@ -108,9 +104,6 @@
     },
     remove: function() {
       scope.removeMulti([this]);
-    },
-    _updateActiveDuration: function() {
-      this._activeDuration = shared.calculateActiveDuration(this._timing);
     }
   };
 

src/keyframe-effect.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(shared, scope, testing) {
 
   function EffectTime(timing) {

src/keyframe-interpolations.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(shared, scope, testing) {
 
   scope.convertEffectInput = function(effectInput) {

src/matrix-decomposition.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
   var decomposeMatrix = (function() {
     function determinant(m) {

src/matrix-interpolation.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
   var composeMatrix = (function() {
     function multiply(a, b) {

src/normalize-keyframes.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(shared, testing) {
   var shorthandToLonghand = {
     background: [

src/number-handler.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
 
   function numberToString(x) {

src/position-handler.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope) {
 
   function negateDimension(dimension) {

src/property-interpolation.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(shared, scope, testing) {
 
   var propertyHandlers = {};

src/property-names.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
 
   var aliased = {};

src/scope.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 var webAnimationsShared = {};
 var webAnimations1 = {};
 var webAnimationsNext = {};

src/shadow-handler.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope) {
 
   function consumeShadow(string) {

src/shape-handler.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope) {
 
   var consumeLengthOrPercent = scope.consumeParenthesised.bind(null, scope.parseLengthOrPercent);

src/tick.js

@@ -12,7 +12,7 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
+
 (function(shared, scope, testing) {
   var originalRequestAnimationFrame = window.requestAnimationFrame;
   var rafCallbacks = [];

src/timeline.js

@@ -12,7 +12,7 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
+
 (function(shared, scope, testing) {
   var originalRequestAnimationFrame = window.requestAnimationFrame;
   window.requestAnimationFrame = function(f) {

src/timing-utilities.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(shared, testing) {
 
   var fills = 'backwards|forwards|both|none'.split('|');
@@ -53,7 +52,7 @@
       if (this._effect) {
         this._effect._timingInput[member] = value;
         this._effect._timing = shared.normalizeTimingInput(this._effect._timingInput);
-        this._effect._updateActiveDuration();
+        this._effect.activeDuration = shared.calculateActiveDuration(this._effect._timing);
         if (this._effect._animation) {
           this._effect._animation._rebuildUnderlyingAnimation();
         }
@@ -144,7 +143,7 @@
           if ((property == 'direction') && (directions.indexOf(timingInput[property]) == -1)) {
             return;
           }
-          if (property == 'playbackRate') {
+          if (property == 'playbackRate' && timingInput[property] !== 1 && shared.isDeprecated('AnimationEffectTiming.playbackRate', '2014-11-28', 'Use Animation.playbackRate instead.')) {
             return;
           }
           timing[property] = timingInput[property];

src/transform-handler.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
 
   // This returns a function for converting transform functions to equivalent

src/visibility-handler.js

@@ -12,7 +12,6 @@
 //   See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(scope, testing) {
 
   function merge(left, right) {

src/web-animations-bonus-cancel-events.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function() {
 
   if (document.createElement('div').animate([]).oncancel !== undefined) {

src/web-animations-bonus-object-form-keyframes.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(shared) {
   // If the polyfill is being loaded in a context where Element.animate is
   // supported but object-form syntax is not, then creating an animation

src/web-animations-next-animation.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function(shared, scope, testing) {
   scope.animationsWithPromises = [];
 

test/testharness-runner.js

@@ -12,7 +12,6 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
-'use strict';
 (function() {
   var assert = chai.assert;
   mocha.setup({ ui: 'tdd' });