Merge pull request #488 from ewilligers/strict

Use strict

src/animation.js

@@ -12,6 +12,7 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
+'use strict';
 (function(shared, scope, testing) {
 
   shared.sequenceNumber = 0;
@@ -191,7 +192,6 @@
       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,6 +12,7 @@
 //     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,6 +12,7 @@
 //     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,6 +12,7 @@
 //   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,6 +12,7 @@
 //   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,6 +12,7 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
+'use strict';
 (function(shared) {
 
   var silenced = {};

src/dev.js

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

src/handler-utils.js

@@ -12,6 +12,7 @@
 //   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,6 +12,7 @@
 //     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,6 +12,7 @@
 //     See the License for the specific language governing permissions and
 // limitations under the License.
 
+'use strict';
 (function(shared, scope, testing) {
 
   var disassociate = function(effect) {
@@ -70,7 +71,7 @@
       this._normalizedKeyframes = new KeyframeList(effectInput);
     }
     this._keyframes = effectInput;
-    this.activeDuration = shared.calculateActiveDuration(this._timing);
+    this._updateActiveDuration();
     this._id = id;
     return this;
   };
@@ -93,6 +94,9 @@
     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.');
@@ -104,6 +108,9 @@
     },
     remove: function() {
       scope.removeMulti([this]);
+    },
+    _updateActiveDuration: function() {
+      this._activeDuration = shared.calculateActiveDuration(this._timing);
     }
   };
 

src/keyframe-effect.js

@@ -12,6 +12,7 @@
 //     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,6 +12,7 @@
 //     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,6 +12,7 @@
 //   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,6 +12,7 @@
 //   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,6 +12,7 @@
 //     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,6 +12,7 @@
 //   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,6 +12,7 @@
 //   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,6 +12,7 @@
 //   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,6 +12,7 @@
 //     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,6 +12,7 @@
 //     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,6 +12,7 @@
 //   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,6 +12,7 @@
 //   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,6 +12,7 @@
 //     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('|');
@@ -52,7 +53,7 @@
       if (this._effect) {
         this._effect._timingInput[member] = value;
         this._effect._timing = shared.normalizeTimingInput(this._effect._timingInput);
-        this._effect.activeDuration = shared.calculateActiveDuration(this._effect._timing);
+        this._effect._updateActiveDuration();
         if (this._effect._animation) {
           this._effect._animation._rebuildUnderlyingAnimation();
         }
@@ -143,7 +144,7 @@
           if ((property == 'direction') && (directions.indexOf(timingInput[property]) == -1)) {
             return;
           }
-          if (property == 'playbackRate' && timingInput[property] !== 1 && shared.isDeprecated('AnimationEffectTiming.playbackRate', '2014-11-28', 'Use Animation.playbackRate instead.')) {
+          if (property == 'playbackRate') {
             return;
           }
           timing[property] = timingInput[property];

src/transform-handler.js

@@ -12,6 +12,7 @@
 //   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,6 +12,7 @@
 //   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,6 +12,7 @@
 //     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,6 +12,7 @@
 //     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,6 +12,7 @@
 //     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,6 +12,7 @@
 //     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' });