Call el.offsetHeight; to force update of computed style.

Call el.offsetHeight; to force update of computed style.

Transitions does not work if change is 'auto' -> '0'. As work around code sets
height to offsetHeight before changing height to '0'. However Blink still uses
computed style and does not starts transition. So we have to call el.offsetHeight
to force update computed style.

Removed unused argument from onFadeOutTransitionEnd.

BUG=330604
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245280

[Noam Samuel, 2014-01-15 22:08:44]

lgtm

https://codereview.chromium.org/139963002/diff/1/chrome/browser/resources/pri...
File chrome/browser/resources/print_preview/print_preview_animations.js (right):

https://codereview.chromium.org/139963002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/print_preview_animations.js:67:
el.offsetHeight;  // Should force an update of the computed style.
Should this have a TODO attached to it to remove it when the corresponding bug
in Blink is fixed?

[Vitaly Buka (NO REVIEWS), 2014-01-15 22:13:54]

On 2014/01/15 22:08:44, Noam Samuel wrote:
> lgtm
> 
>
https://codereview.chromium.org/139963002/diff/1/chrome/browser/resources/pri...
> File chrome/browser/resources/print_preview/print_preview_animations.js
(right):
> 
>
https://codereview.chromium.org/139963002/diff/1/chrome/browser/resources/pri...
> chrome/browser/resources/print_preview/print_preview_animations.js:67:
> el.offsetHeight;  // Should force an update of the computed style.
> Should this have a TODO attached to it to remove it when the corresponding bug
> in Blink is fixed?

According dstockwell it's intended behavior.

[commit-bot: I haz the power, 2014-01-15 22:17:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vitalybuka@chromium.org/139963002/1

[Noam Samuel, 2014-01-15 22:19:24]

On 2014/01/15 22:13:54, Vitaly Buka wrote:
> On 2014/01/15 22:08:44, Noam Samuel wrote:
> > lgtm
> > 
> >
>
https://codereview.chromium.org/139963002/diff/1/chrome/browser/resources/pri...
> > File chrome/browser/resources/print_preview/print_preview_animations.js
> (right):
> > 
> >
>
https://codereview.chromium.org/139963002/diff/1/chrome/browser/resources/pri...
> > chrome/browser/resources/print_preview/print_preview_animations.js:67:
> > el.offsetHeight;  // Should force an update of the computed style.
> > Should this have a TODO attached to it to remove it when the corresponding
bug
> > in Blink is fixed?
> 
> According dstockwell it's intended behavior.

lgtm++

[commit-bot: I haz the power, 2014-01-16 19:32:12]

Change committed as 245280