Bug 1782339 Comment 13 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
(In reply to Hiroyuki Ikezoe (:hiro) from comment #12)
> The code I meant is [here](https://searchfox.org/mozilla-central/rev/c454f13d83dc85f399fd1eb449ea9ccb156299df/gfx/layers/apz/src/AsyncPanZoomController.cpp#3529);
> 
> ```
> aStartPoint = aEndPoint + overscroll
> ```

Isn't [this check](https://searchfox.org/mozilla-central/rev/c454f13d83dc85f399fd1eb449ea9ccb156299df/gfx/layers/apz/src/AsyncPanZoomController.cpp#3542) sufficient?
Revision 1 by
on 
(In reply to Hiroyuki Ikezoe (:hiro) from comment #12)
> The code I meant is [here](https://searchfox.org/mozilla-central/rev/c454f13d83dc85f399fd1eb449ea9ccb156299df/gfx/layers/apz/src/AsyncPanZoomController.cpp#3529);
> 
> ```
> aStartPoint = aEndPoint + overscroll
> ```

Isn't [this check](https://searchfox.org/mozilla-central/rev/c454f13d83dc85f399fd1eb449ea9ccb156299df/gfx/layers/apz/src/AsyncPanZoomController.cpp#3542) sufficient?

EDIT: Or I guess you mean `overscroll` should be checked before assigning the output parameter `aStartPoint`?

Back to Bug 1782339 Comment 13