Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java |
| index aec39b8f0126197ad9cd8df2e047acf0191d4779..f7217f790dd91bb91f2db0996d2d842378bcf8f2 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java |
| @@ -10,6 +10,7 @@ import android.graphics.Bitmap; |
| import org.chromium.base.ContextUtils; |
| import org.chromium.base.ThreadUtils; |
| +import org.chromium.base.metrics.RecordUserAction; |
| import org.chromium.chrome.browser.UrlConstants; |
| import org.chromium.chrome.browser.favicon.FaviconHelper; |
| import org.chromium.chrome.browser.favicon.FaviconHelper.FaviconImageCallback; |
| @@ -210,7 +211,7 @@ public class RecentTabsManager implements AndroidSyncSettingsObserver, SignInSta |
| public void openForeignSessionTab(ForeignSession session, ForeignSessionTab tab, |
| int windowDisposition) { |
| if (mIsDestroyed) return; |
| - NewTabPageUma.recordAction(NewTabPageUma.ACTION_OPENED_FOREIGN_SESSION); |
|
dgn
2016/12/22 12:31:22
Isn't that constant now unused? It could be remove
finkm
2017/01/03 10:43:52
Done.
|
| + RecordUserAction.record("MobileTabFromOtherDeviceOpen"); |
| mForeignSessionHelper.openForeignSessionTab(mTab, session, tab, windowDisposition); |
| } |
| @@ -223,7 +224,7 @@ public class RecentTabsManager implements AndroidSyncSettingsObserver, SignInSta |
| */ |
| public void openRecentlyClosedTab(RecentlyClosedTab tab, int windowDisposition) { |
| if (mIsDestroyed) return; |
| - NewTabPageUma.recordAction(NewTabPageUma.ACTION_OPENED_RECENTLY_CLOSED_ENTRY); |
| + RecordUserAction.record("MobileRecentlyClosedTabOpen"); |
| mRecentlyClosedBridge.openRecentlyClosedTab(mTab, tab, windowDisposition); |
| } |