Skip to content
Permalink
Browse files

No ticket. Remove the unnecessary guard in addGetHookIf. Close gh-1426.

In 1.x if the support test executes before doc ready, it may not be able
to return a result yet. In such a case, we protect ourselves from future
breakages, allowing only for the ones before doc ready.

Since in 2.x lazy support tests attach test elements to docElem, not body,
such a guard is unnecessary.
  • Loading branch information...
mgol committed Nov 11, 2013
1 parent 4b39a82 commit 07c374745790bf9e05ca58ffe97f0b72ae930e66
Showing with 1 addition and 9 deletions.
  1. +1 −9 src/css/addGetHookIf.js
@@ -4,15 +4,7 @@ function addGetHookIf( conditionFn, hookFn ) {
// Define the hook, we'll check on the first run if it's really needed.
return {
get: function() {
var condition = conditionFn();

if ( condition == null ) {
// The test was not ready at this point; screw the hook this time
// but check again when needed next time.
return;
}

if ( condition ) {
if ( conditionFn() ) {
// Hook not needed (or it's not possible to use it due to missing dependency),
// remove it.
// Since there are no other hooks for marginRight, remove the whole object.

0 comments on commit 07c3747

Please sign in to comment.
You can’t perform that action at this time.