(perf) - move the collapser check before inling properties

[vigneshshanmugam]

This yields us a huge 3.5x improvement on a test file > 2MB.

pluginAlias                       time(ms)

// Before
transform-inline-consecutive-adds - 5716.129
transform-inline-consecutive-adds - 5752.153
transform-inline-consecutive-adds - 5813.064

// After
transform-inline-consecutive-adds - 1624.955
transform-inline-consecutive-adds - 1925.298
transform-inline-consecutive-adds - 1788.473

Thanks to chrome devtools for helping to find the root cause. Confirmation from devtools

Before

After

[sophiebits]

.get('body') creating an array seems like a huge performance trap @hzoo. Same thing I ran into in babel/babel#7028.

[vigneshshanmugam]

Thanks @sophiebits for pointing out. Good to know. I am also seeing smaller perf slowdown on get calls from path not just on 'body'. It happens on all array elements

Probably it has something to do with https://github.com/babel/babel/blob/068c65a1dd5bf606d0441506f2f5773be168223b/packages/babel-traverse/src/path/family.js#L104

[sophiebits]

Yes.

[boopathi]

Awesome!.