Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(202)

Issue 23534019: Add Quickoffice UMA histograms for spreadsheets. (Closed)

Created:
7 years, 3 months ago by dusk
Modified:
7 years, 3 months ago
CC:
chromium-reviews, Ilya Sherman, jar (doing other things), asvitkine+watch_chromium.org
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Add Quickoffice UMA histograms for spreadsheets. Added histogram definitions for 2 new data points from the Quickoffice spreadsheet extension - fomattedCellCount will log the total number of cells in the default worksheet that have any formatting content. This will allow us to better understand typical spreadsheet use and schedule performance work accordingly. One histogram tracks compound binary format spreadsheets, the other OOXML spreadsheets. BUG=276229 NOTRY=true Change-Id: I3ef3b81eade4916447daac5aa281c0eda2a3694f Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=220421

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -0 lines) Patch
M tools/metrics/histograms/histograms.xml View 2 chunks +14 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
dusk
Ready for review.
7 years, 3 months ago (2013-08-29 07:07:14 UTC) #1
Ilya Sherman
LGTM.
7 years, 3 months ago (2013-08-29 23:09:57 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dskelton@chromium.org/23534019/1
7 years, 3 months ago (2013-08-29 23:11:48 UTC) #3
commit-bot: I haz the power
7 years, 3 months ago (2013-08-29 23:13:09 UTC) #4
Message was sent while issue was closed.
Change committed as 220421

Powered by Google App Engine
This is Rietveld 408576698