Gather statistics for what percentage of document views include the following user actions: - The document is printed - The document is downloaded
Also: - The bookmarks bar is opened - Any bookmark is followed
Also: - Change page by typing the number on the toolbar - Rotate - Fit to width - Fit to page
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ffb238499e13834daa8c0f11d78b9ea8ce95598d commit ffb238499e13834daa8c0f11d78b9ea8ce95598d Author: Henrique Nakashima <hnakashima@chromium.org> Date: Tue Dec 05 21:53:40 2017 Whitelist PDF Viewer to use metricsPrivate Bug: 768982 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I030f371dcbc1fbef3bcc01ce1daa110cd5b6ad1f Reviewed-on: https://chromium-review.googlesource.com/807144 Reviewed-by: Devlin <rdevlin.cronin@chromium.org> Reviewed-by: dsinclair <dsinclair@chromium.org> Commit-Queue: Henrique Nakashima <hnakashima@chromium.org> Cr-Commit-Position: refs/heads/master@{#521845} [modify] https://crrev.com/ffb238499e13834daa8c0f11d78b9ea8ce95598d/chrome/browser/resources/pdf/manifest.json [modify] https://crrev.com/ffb238499e13834daa8c0f11d78b9ea8ce95598d/extensions/common/api/_permission_features.json
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a commit 9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a Author: Henrique Nakashima <hnakashima@chromium.org> Date: Mon Dec 11 19:16:49 2017 Metrics for user commands in PDF viewer. The following metrics are added: - The bookmarks panel is opened - Any bookmark is followed - Change page by typing the number on the toolbar - Rotate - Fit to width - Fit to page We need these metrics to make informed decisions about tweaks to our current UI and an upcoming overhaul. Bug: 768982 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ia871b4d390ea7dc28f9017780c9622fb5d54cbce Reviewed-on: https://chromium-review.googlesource.com/801634 Commit-Queue: Henrique Nakashima <hnakashima@chromium.org> Reviewed-by: dsinclair <dsinclair@chromium.org> Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by: Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#523164} [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/browser/pdf/pdf_extension_test.cc [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/browser/resources/component_extension_resources.grd [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/browser/resources/pdf/elements/viewer-bookmark/viewer-bookmark.js [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/browser/resources/pdf/elements/viewer-page-selector/viewer-page-selector.js [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/browser/resources/pdf/elements/viewer-pdf-toolbar/viewer-pdf-toolbar.html [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/browser/resources/pdf/elements/viewer-toolbar-dropdown/viewer-toolbar-dropdown.js [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/browser/resources/pdf/elements/viewer-zoom-toolbar/viewer-zoom-toolbar.js [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/browser/resources/pdf/index.html [add] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/browser/resources/pdf/metrics.js [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/browser/resources/pdf/pdf.js [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/test/data/pdf/material_elements_test.js [add] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/chrome/test/data/pdf/metrics_test.js [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/tools/metrics/histograms/enums.xml [modify] https://crrev.com/9e68ea0bc435e2b8bf9a03ef6bd165618a93c92a/tools/metrics/histograms/histograms.xml
Still missing metrics for printing and downloading, since they can come from the browser. If we figure how to handle them, it would be useful for saving too (crbug.com/61248).
Comment 1 by hnakashima@chromium.org
, Sep 26 2017