Fix dumpToLocalStorate guard race condition

[linstula]

• Remove isNotEmpty guard for _dumpToLocalStorage becuase we want localStorage to be updated if we remove items from the cart via removeItem.

[bcardarella]

What's the thinking behind removing on('init')?

[linstula]

It seems that when an ArrayProxy is initialized, it initialized with an empty array as the content. When the observer fires on init, its basically overwriting localStorage to an empty cart, so, it'll blow your cart away if you had stuff in the cart on init.

[bcardarella]

I think that is a race condition issue then, one that can be addressed in the initializer.