| 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);
|
| }
|
| };
|
|
|