requestAnimationFrame should return native handleId or cancelAF should accept TimerOption

[lacolaco]

From angular/angular#8401

Currently, zone-patched requestAnimationFrame returns ZoneTask.

It seems breaking native rAF canceling:

let handleId = requestAnimationFrame(fn);
cancelAnimationFrame(handleId); // not canceled

To cancel rAF, we have to use data.handleId

let timerOption = requestAnimationFrame(fn);
cancelAnimationFrame(timerOption.data.handleId);

[lacolaco]

Repro (with Angular2): http://plnkr.co/edit/MmQmT8YAbOyg8kh3pnQX?p=preview

[sebholstein]

Maybe related to #359

[b-strauss]

I think zone.js should not change the signature of any native APIs. Is there a list of all patches that do not preserve the original signature?

[lacolaco]

AFAIK, set/clearInterval and set/clearTimeout seems working as well as its native behavior.

[lacolaco]

@IgorMinar how do you think about this?

[lacolaco]

Fixed in latest zone.js. http://plnkr.co/edit/JJCleR?p=preview
Thank you