| commit | f2b477db48c7e330caa0c665c064d60dc2599032 | |
| author | dominikg@chromium.org <dominikg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Wed, 4 Dec 2013 20:15:48 +0000 (4 20:15 +0000) | ||
| committer | dominikg@chromium.org <dominikg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Wed, 4 Dec 2013 20:15:48 +0000 (4 20:15 +0000) | ||
| tree | a78b449dd5ac789e47bb889d828a109f07719bd9 | treesnapshot (tar.gz zip) |
| parent | b04ddb4894dba9e78634de7d6d63bce00d749927 | commitdiff |
Telemetry: Start scrolling at current position.
The scroll action would set scrollTop to 0 before scrolling, so we always start
from the top of the page. For pages that require multiple scrolls, e.g. because
more content is loaded dynamically when the bottom is reached, we want to scroll
from the current position rather than going back to the top.
In the typical case we only perform one scroll immediately after the page was
loaded, so this change shouldn't affect any measurements.
BUG=299828
Review URL: https://codereview.chromium.org/101323004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238748 0039d316-1c4b-4281-b951-d872f2087c98
The scroll action would set scrollTop to 0 before scrolling, so we always start
from the top of the page. For pages that require multiple scrolls, e.g. because
more content is loaded dynamically when the bottom is reached, we want to scroll
from the current position rather than going back to the top.
In the typical case we only perform one scroll immediately after the page was
loaded, so this change shouldn't affect any measurements.
BUG=299828
Review URL: https://codereview.chromium.org/101323004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238748 0039d316-1c4b-4281-b951-d872f2087c98
| tools/telemetry/telemetry/page/actions/scroll.js | diffblobblamehistory |