Commit 0b12ba04 authored by Daniel Murphy's avatar Daniel Murphy Committed by Commit Bot

[ManageSpace] Record histogram of user's web storage size

Bug: 875439
Change-Id: If6c56a4c634ce685a4e1ab93660ae95db7d90716
Reviewed-on: https://chromium-review.googlesource.com/c/1274599Reviewed-by: default avatarMark Pearson <mpearson@chromium.org>
Reviewed-by: default avatarTheresa <twellington@chromium.org>
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599380}
parent a4db0e4a
...@@ -40,6 +40,7 @@ import org.chromium.chrome.browser.preferences.Preferences; ...@@ -40,6 +40,7 @@ import org.chromium.chrome.browser.preferences.Preferences;
import org.chromium.chrome.browser.preferences.PreferencesLauncher; import org.chromium.chrome.browser.preferences.PreferencesLauncher;
import org.chromium.chrome.browser.preferences.website.Website.StoredDataClearedCallback; import org.chromium.chrome.browser.preferences.website.Website.StoredDataClearedCallback;
import org.chromium.chrome.browser.searchwidget.SearchWidgetProvider; import org.chromium.chrome.browser.searchwidget.SearchWidgetProvider;
import org.chromium.chrome.browser.util.ConversionUtils;
import java.util.Collection; import java.util.Collection;
...@@ -274,6 +275,10 @@ public class ManageSpaceActivity extends AppCompatActivity implements View.OnCli ...@@ -274,6 +275,10 @@ public class ManageSpaceActivity extends AppCompatActivity implements View.OnCli
} }
private void onSiteStorageSizeCalculated(long totalSize, long unimportantSize) { private void onSiteStorageSizeCalculated(long totalSize, long unimportantSize) {
RecordHistogram.recordCountHistogram("Android.ManageSpace.TotalDiskUsageMB",
(int) ConversionUtils.bytesToMegabytes(totalSize));
RecordHistogram.recordCountHistogram("Android.ManageSpace.UnimportantDiskUsageMB",
(int) ConversionUtils.bytesToMegabytes(unimportantSize));
mSiteDataSizeText.setText(Formatter.formatFileSize(this, totalSize)); mSiteDataSizeText.setText(Formatter.formatFileSize(this, totalSize));
mUnimportantSiteDataSizeText.setText(Formatter.formatFileSize(this, unimportantSize)); mUnimportantSiteDataSizeText.setText(Formatter.formatFileSize(this, unimportantSize));
} }
......
...@@ -1939,6 +1939,24 @@ uploading your change for review. ...@@ -1939,6 +1939,24 @@ uploading your change for review.
</summary> </summary>
</histogram> </histogram>
<histogram name="Android.ManageSpace.TotalDiskUsageMB" units="MB">
<owner>dmurph@chromium.org</owner>
<summary>
Recorded when the user opens the 'Manage Space' screen for Chrome, after the
disk usage is calculated. The disk usage value represents the accumulated
disk usage by websites.
</summary>
</histogram>
<histogram name="Android.ManageSpace.UnimportantDiskUsageMB" units="MB">
<owner>dmurph@chromium.org</owner>
<summary>
Recorded when the user opens the 'Manage Space' screen for Chrome, after the
disk usage is calculated. The disk usage value represents the accumulated
disk usage by websites that are not considered 'important'.
</summary>
</histogram>
<histogram name="Android.MemoryPressureMonitor.GetMyMemoryState.Failed.Time" <histogram name="Android.MemoryPressureMonitor.GetMyMemoryState.Failed.Time"
units="microseconds"> units="microseconds">
<owner>dskiba@chromium.org</owner> <owner>dskiba@chromium.org</owner>
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment