Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java |
| index fe2af4fd64eeeba0a5007f1e79643eb582f19c82..21db4502bd4f6395565875acc2318101a8ca007d 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java |
| @@ -11,6 +11,7 @@ import android.graphics.drawable.Drawable; |
| import android.net.Uri; |
| import android.os.Bundle; |
| import android.os.StrictMode; |
| +import android.os.SystemClock; |
| import android.text.TextUtils; |
| import android.view.LayoutInflater; |
| import android.view.View; |
| @@ -25,6 +26,7 @@ import org.chromium.base.ApplicationStatus; |
| import org.chromium.base.Log; |
| import org.chromium.base.StreamUtil; |
| import org.chromium.base.VisibleForTesting; |
| +import org.chromium.base.metrics.RecordHistogram; |
| import org.chromium.blink_public.platform.WebDisplayMode; |
| import org.chromium.chrome.R; |
| import org.chromium.chrome.browser.TabState; |
| @@ -48,6 +50,7 @@ import java.io.File; |
| import java.io.FileNotFoundException; |
| import java.io.FileOutputStream; |
| import java.io.IOException; |
| +import java.util.concurrent.TimeUnit; |
| /** |
| * Displays a webapp in a nearly UI-less Chrome (InfoBars still appear). |
| @@ -169,8 +172,11 @@ public class WebappActivity extends FullScreenActivity { |
| StrictMode.ThreadPolicy oldPolicy = StrictMode.allowThreadDiskReads(); |
| StrictMode.allowThreadDiskWrites(); |
| try { |
| + long time = SystemClock.elapsedRealtime(); |
| foutput = new FileOutputStream(tabFile); |
| TabState.saveState(foutput, getActivityTab().getState(), false); |
| + RecordHistogram.recordTimesHistogram("Android.StrictMode.WebappSaveState", |
| + SystemClock.elapsedRealtime() - time, TimeUnit.MILLISECONDS); |
| } catch (FileNotFoundException exception) { |
| Log.e(TAG, "Failed to save out tab state.", exception); |
| } catch (IOException exception) { |
| @@ -289,7 +295,7 @@ public class WebappActivity extends FullScreenActivity { |
| } |
| mWebappUma.recordSplashscreenIconType(splashScreenIconType); |
| mWebappUma.recordSplashscreenIconSize( |
| - Math.round((float) displayIcon.getWidth() |
|
Yaron
2016/01/21 20:48:15
was this removal intentional?
Peter Wen
2016/01/21 20:59:45
Yeah, eclipse auto-removed it since density is a f
|
| + Math.round(displayIcon.getWidth() |
| / getResources().getDisplayMetrics().density)); |
| } |