diff options
author | Yorke Lee <yorkelee@google.com> | 2015-11-06 10:35:30 -0800 |
---|---|---|
committer | Yorke Lee <yorkelee@google.com> | 2015-11-09 11:18:38 -0800 |
commit | 58e68a9513495cbe76188d6066c4ffa467313175 (patch) | |
tree | 4bf102afd0b4d16d7307236d1f88ea727537c1dc /src/com/android/dialer/DialtactsActivity.java | |
parent | 6e0fd6b77759baf43cb37d407041be051b4856b5 (diff) |
More work to hook up screen event logging
* Change logScreenView to log by an integer rather than string
representing a screen name
* Add some missing log events (SmartDialSearchFragment, etc)
* Add mapping of integer to screen name strings for AnalyticsUtil
Bug: 23164804
Change-Id: If34a01b71a3fb598caa237e4b679cfeb30c4dd01
Diffstat (limited to 'src/com/android/dialer/DialtactsActivity.java')
-rw-r--r-- | src/com/android/dialer/DialtactsActivity.java | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/com/android/dialer/DialtactsActivity.java b/src/com/android/dialer/DialtactsActivity.java index 4fa4bc2d7..cc457bd6c 100644 --- a/src/com/android/dialer/DialtactsActivity.java +++ b/src/com/android/dialer/DialtactsActivity.java @@ -82,6 +82,7 @@ import com.android.dialer.list.SearchFragment; import com.android.dialer.list.SmartDialSearchFragment; import com.android.dialer.list.SpeedDialFragment; import com.android.dialer.logging.Logger; +import com.android.dialer.logging.ScreenEvent; import com.android.dialer.settings.DialerSettingsActivity; import com.android.dialer.util.DialerUtils; import com.android.dialer.util.IntentUtil; @@ -527,7 +528,7 @@ public class DialtactsActivity extends TransactionSafeActivity implements View.O // This is only called when the activity goes from resumed -> paused -> resumed, so it // will not cause an extra view to be sent out on rotation if (mIsDialpadShown) { - Logger.logFragmentView(mDialpadFragment); + Logger.logScreenView(ScreenEvent.DIALPAD, this); } mIsRestarting = false; } @@ -682,12 +683,15 @@ public class DialtactsActivity extends TransactionSafeActivity implements View.O ImportExportDialogFragment.show(getFragmentManager(), true, DialtactsActivity.class, ImportExportDialogFragment.EXPORT_MODE_DEFAULT); } + Logger.logScreenView(ScreenEvent.IMPORT_EXPORT_CONTACTS, this); return true; case R.id.menu_clear_frequents: ClearFrequentsDialog.show(getFragmentManager()); + Logger.logScreenView(ScreenEvent.CLEAR_FREQUENTS, this); return true; case R.id.menu_call_settings: handleMenuSettings(); + Logger.logScreenView(ScreenEvent.SETTINGS, this); return true; } return false; @@ -742,9 +746,7 @@ public class DialtactsActivity extends TransactionSafeActivity implements View.O } mDialpadFragment.setAnimate(animate); - // logScreenView is used here explicitly to provide the activity as the DialpadFragment - // might not have been attached yet. - Logger.logScreenView(DialpadFragment.class.getSimpleName(), this, null); + Logger.logScreenView(ScreenEvent.DIALPAD, this); ft.commit(); if (animate) { |