MetricsService "idleness" concept can be defeated by a page that auto reloads. |
|
Issue descriptionMetricsService "idleness" concept can be defeated by a page that auto reloads. For example, if you open the attached HTML page that just has a JS timer to auto-reload, it will keep issuing some UMA actions - namely NavEntryCommitted/FrameLoad. Since we use presence of an action to control whether MetricsService is idle, it means a user having such a page open will result in continuous UMA uploads even if they're AFK. One solution would be to stop logging those two user actions, although there may be other such cases. We can also look into hooking into the "desktop engagement" metric mechanism ( crbug.com/615080 ), once that materialized.
,
Aug 11 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e2be21dbfe73a8e41cd980504567b002d4e03222 commit e2be21dbfe73a8e41cd980504567b002d4e03222 Author: asvitkine <asvitkine@chromium.org> Date: Thu Aug 11 15:57:51 2016 Remove a number of non-user triggered UMA actions. Specifically, removes the following UMA actions: FrameLoad, FrameLoadWithFlash, The non-suffixed version would be logged by every page load (including pages that use a script to auto reload themselves) and is thus pretty spammy. It also seems like the kind of info they were tracking would be better suited to track with enum histograms. These could be added if there's a desire to keep these stats. BUG=624504 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation Review-Url: https://codereview.chromium.org/2226083002 Cr-Commit-Position: refs/heads/master@{#411339} [modify] https://crrev.com/e2be21dbfe73a8e41cd980504567b002d4e03222/content/browser/frame_host/navigator_impl.cc [modify] https://crrev.com/e2be21dbfe73a8e41cd980504567b002d4e03222/content/renderer/render_frame_impl.cc [modify] https://crrev.com/e2be21dbfe73a8e41cd980504567b002d4e03222/tools/metrics/actions/actions.xml
,
Jan 11
You started fixing this bug over two years ago. Are you still working on it? |
|
►
Sign in to add a comment |
|
Comment 1 by asvitk...@chromium.org
, Jun 29 2016360 bytes
360 bytes View Download