Revert of [telemetry] Bullet proof scroll.js (patchset #4 id:60001 of https://codereview.chromium.org/2532443002/ )
Revert of [telemetry] Bullet proof scroll.js (patchset #4 id:60001 of https://codereview.chromium.org/2532443002/ )

Reason for revert:
CL breaks catapult->chrome autoroller see: https://github.com/catapult-project/catapult/issues/3149

Original issue's description:
> [telemetry] Bullet proof scroll.js
>
> Make scroll.js give a nice error message if the scroll point is
> off screen or if the element has zero area. Previously we just
> got a stack trace from chrome.
>
> Also fix various lint errors since this is the first time this file
> has been touched in a while.
>
> BUG=chromium:664515
>
> Review-Url: https://codereview.chromium.org/2532443002
> Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/c3ec514d8726530643844182677304ce7f15a5c2

TBR=skyostil@chromium.org,perezju@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:664515
NOPRESUBMIT=true
NOTRY=true

Review-Url: https://codereview.chromium.org/2644073003

sahiljain/catapult,catapult-project/catapult,catapult-project/catapult-csm,benschmaus/catapult,benschmaus/catapult,catapult-project/catapult-csm,catapult-project/catapult,catapult-project/catapult,catapult-project/catapult-csm,catapult-project/catapult,catapult-project/catapult-csm,catapult-project/catapult,catapult-project/catapult,sahiljain/catapult,catapult-project/catapult-csm,benschmaus/catapult,sahiljain/catapult,benschmaus/catapult,sahiljain/catapult,sahiljain/catapult,benschmaus/catapult,benschmaus/catapult,catapult-project/catapult-csm,sahiljain/catapult,catapult-project/catapult-csm,catapult-project/catapult,benschmaus/catapult