better handling of setTimeout in mobile

[ben hockey]

the setTimeout calls used extensively throughout dojox/mobile (particularly the longer ones) should be tracked and if there are any outstanding timeouts when a widget is destroyed, those should be cleared. _WidgetBase has a defer function to help with this that might be useful.

in particular, there is one specific call in _ItemBase that is constantly causing problems in an app i'm working on when the widgets are destroyed before that setTimeout completes:

setTimeout(function(){
        _this.set("selected", false);
}, this._duration);

this._duration is 800ms by default and by the time that function is called, the domNode has been set to null and causes all kinds of problems. this setTimeout should be cleared in the destroy of the widget.

[bill]

You should use _WidgetBase::defer() (added by Doug).

[cjolif]

I think this one was fixed to use defer(). Eric can you check if we have other ones that should use defer() and are not? And if yes I think it would be good to have them fixed in 1.9.

[Adrian Vasiliu]

Use of widget.defer() instead of setTimeout. This fixes some cases of failures when destroying the widget. Includes DOH testing. - Adrian Vasiliu (IBM, CCLA)

[cjolif]

In [30907]:

fixes #15064. Use of widget.defer() instead of setTimeout. This fixes some cases of failures when destroying the widget. Thanks Adrian Vasiliu (IBM, CCLA). !strict

[cjolif]

In [30908]:

fixes #15064. Use of widget.defer() instead of setTimeout. This fixes some cases of failures when destroying the widget. Test cases. Thanks Adrian Vasiliu (IBM, CCLA). !strict.

[Adrian Vasiliu]

Fix in previous patch (removal of widget.defer) - Adrian Vasiliu (IBM, CCLA)

[cjolif]

In [30918]:

refs #15064. properly remove the timeout now that they are fire by this.defer(). Thanks Adrian Vasiliu (IBM, CCLA)

[Adrian Vasiliu]

Remove unnecessary lang.hitch when using widget.defer (done anyway internally by widget.defer) - Adrian Vasiliu (IBM, CCLA)

[Patrick Ruzand]

In [30920]:

Remove unnecessary lang.hitch when using widget.defer, fixes #15064, thx Adrian Vasiliu (CCLA) !strict

[Adrian Vasiliu]

scrollable: do not use widget.defer() because this module is not always mixed into a widget - Adrian Vasiliu (IBM, CCLA)

[Adrian Vasiliu]

(committed in [30956])