Skip to content

Commit

Permalink
Fixing comments to pull request
Browse files Browse the repository at this point in the history
  • Loading branch information
anenviousguest committed Sep 3, 2014
1 parent 4e42ae4 commit 3e25189
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
10 changes: 5 additions & 5 deletions src/createElement.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
define(function() {
var createElement = function() {
if (typeof document.createElement !== 'function') {
var oldCreateElement = document.createElement;
document.createElement = function (tagName) {
return oldCreateElement(tagName);
};
// This is the case in IE7, where the type of createElement is "object".
// For this reason, we cannot call apply() as Object is not a Function.
return document.createElement(arguments[0]);
} else {
return document.createElement.apply(document, arguments);
}
return document.createElement.apply(document, arguments);
};
return createElement;
});
5 changes: 3 additions & 2 deletions src/testProps.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ define(['contains', 'mStyle', 'createElement', 'nativeTestProps', 'is', 'cssToDO
}

// Otherwise do it properly
var afterInit, i, prop, before;
var afterInit, i, propsLength, prop, before;

// If we don't have a style element, that means
// we're running async or after the core tests,
Expand All @@ -44,7 +44,8 @@ define(['contains', 'mStyle', 'createElement', 'nativeTestProps', 'is', 'cssToDO
}
}

for ( i = 0; i < props.length; i++ ) {
propsLength = props.length;
for ( i = 0; i < propsLength; i++ ) {
prop = props[i];
before = mStyle.style[prop];

Expand Down

0 comments on commit 3e25189

Please sign in to comment.