Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoDelegateImpl.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoDelegateImpl.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoDelegateImpl.java |
index d38337333bab4fe38381f1fccce40f6e2d2b6d16..c5d225c0a7752916dc3abd72f70b29858f450199 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoDelegateImpl.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoDelegateImpl.java |
@@ -38,7 +38,7 @@ public class LogoDelegateImpl implements LogoView.Delegate { |
private String mOnLogoClickUrl; |
private String mAnimatedLogoUrl; |
- private boolean mHasRecordedLoadTime; |
+ private boolean mShouldRecordLoadTime = true; |
private boolean mIsDestroyed; |
@@ -94,13 +94,15 @@ public class LogoDelegateImpl implements LogoView.Delegate { |
if (logo != null) { |
RecordHistogram.recordSparseSlowlyHistogram(LOGO_SHOWN_UMA_NAME, |
logo.animatedLogoUrl == null ? STATIC_LOGO_SHOWN : CTA_IMAGE_SHOWN); |
- if (!mHasRecordedLoadTime) { |
+ if (mShouldRecordLoadTime) { |
long loadTime = System.currentTimeMillis() - loadTimeStart; |
RecordHistogram.recordMediumTimesHistogram( |
LOGO_SHOWN_TIME_UMA_NAME, loadTime, TimeUnit.MILLISECONDS); |
- mHasRecordedLoadTime = true; |
} |
} |
+ // If there currently is no Doodle, don't record the time if a refresh happens |
+ // later. |
+ mShouldRecordLoadTime = false; |
logoObserver.onLogoAvailable(logo, fromCache); |
} |
}; |