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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java

Issue 2915863002: Photo Picker dialog: Add UMA statistics. (Closed)
Patch Set: Address feedback from Mark Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/photo_picker/BitmapScalerTask.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java b/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java
index 5f0b8cb8ae8ea09c898ded533fb097f6e96e3651..56e225b08dab8b384b6cf4474975035f043df9cc 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java
@@ -50,6 +50,7 @@ import org.chromium.chrome.browser.profiles.Profile;
import org.chromium.chrome.browser.tab.Tab;
import org.chromium.chrome.browser.tabmodel.TabModel.TabLaunchType;
import org.chromium.chrome.browser.tabmodel.document.TabDelegate;
+import org.chromium.chrome.browser.util.ConversionUtils;
import org.chromium.chrome.browser.util.IntentUtils;
import org.chromium.components.feature_engagement_tracker.EventConstants;
import org.chromium.components.feature_engagement_tracker.FeatureEngagementTracker;
@@ -94,10 +95,6 @@ public class DownloadUtils {
private static final String EXTRA_IS_OFF_THE_RECORD =
"org.chromium.chrome.browser.download.IS_OFF_THE_RECORD";
- private static final long BYTES_PER_KILOBYTE = 1024;
- private static final long BYTES_PER_MEGABYTE = 1024 * 1024;
- private static final long BYTES_PER_GIGABYTE = 1024 * 1024 * 1024;
-
@VisibleForTesting
static final long SECONDS_PER_MINUTE = TimeUnit.MINUTES.toSeconds(1);
@VisibleForTesting
@@ -769,15 +766,15 @@ public class DownloadUtils {
int resourceId;
float bytesInCorrectUnits;
- if (bytes < BYTES_PER_MEGABYTE) {
+ if (ConversionUtils.bytesToMegabytes(bytes) < 1) {
resourceId = stringSet[0];
- bytesInCorrectUnits = bytes / (float) BYTES_PER_KILOBYTE;
- } else if (bytes < BYTES_PER_GIGABYTE) {
+ bytesInCorrectUnits = bytes / (float) ConversionUtils.BYTES_PER_KILOBYTE;
+ } else if (ConversionUtils.bytesToGigabytes(bytes) < 1) {
resourceId = stringSet[1];
- bytesInCorrectUnits = bytes / (float) BYTES_PER_MEGABYTE;
+ bytesInCorrectUnits = bytes / (float) ConversionUtils.BYTES_PER_MEGABYTE;
} else {
resourceId = stringSet[2];
- bytesInCorrectUnits = bytes / (float) BYTES_PER_GIGABYTE;
+ bytesInCorrectUnits = bytes / (float) ConversionUtils.BYTES_PER_GIGABYTE;
}
return context.getResources().getString(resourceId, bytesInCorrectUnits);
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/photo_picker/BitmapScalerTask.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698