Remove unnecessary cast

[gberaudo]

ERR! compile closure/goog/fx/dragger.js:231: ERROR - unnecessary cast
ERR! compile from: Element
ERR! compile to : (Element|null)
ERR! compile var clonedEl = /** @type {Element} */ (sourceEl.cloneNode(true)),

[gberaudo]

Travis is unhappy due to some errors caused by the compiler.
With an older compiler version it passes at my place.

ERROR - Cannot read: !**\./node_modules**.js
...
12 error(s), 0 warning(s)
The command "./scripts/ci/compile_closure.sh" exited with 12.

[gberaudo]

I bisected the closure compiler and found the first failing commit to be
62ca536d0a184a8852646e2bc2db178f700f5cfc Fix the bug that jscompiler does not respect relative order of --js to --zip inputs.

[Dominator008]

@gberaudo This is a known bug google/closure-compiler#1407
I have a fix but it's not merged yet.

[gberaudo]

Thanks @Dominator008. Looking forward for the merging of your fix.

[joeltine]

The compiler fix was merged.

[gberaudo]

Thanks @joeltine and @Dominator008.
I tested the pending PR google/closure-compiler#1469 and it fixes the build.

[Dominator008]

@gberaudo Good to know. Thanks for testing it out!

[gberaudo]

@joeltine, @Dominator008, just restarted the build but it still fails on travis.
That is strange since on my machine, with the same version of the compiler, a call toscripts/ci/compile_closure.sh works fine. Any idea of what could be wrong?

[joeltine]

Honestly, not a clue atm.

Were you able to repro the breakage on your machine at any point in the past?

[joeltine]

Mmkay I figured it out. It was only the node_modules glob that was broken. I have a fix here: #642.

My guess is that you didn't have any node modules installed in your closure directory and that's why your build succeeded.

[joeltine]

The latest travis failure is unrelated to this change. It was fixed in b21ef38.

Can you please rebase and kick off another Travis run? If it's good, I'll merge this.

[gberaudo]

Travis is passing now, thanks @joeltine.