From b35b0e20a9902a9c39d2c7ed7078e05d60ee633b Mon Sep 17 00:00:00 2001 From: roldenburg Date: Mon, 18 Dec 2017 15:58:29 -0800 Subject: Ensure we do not have a null data string before checking for Duo Bug: 70794242 Test: GoogleCallLogAdapterTest PiperOrigin-RevId: 179480604 Change-Id: Ic53ca21fb7e81390f45b116677df689d3d8fdb67 --- java/com/android/dialer/app/calllog/CallLogListItemViewHolder.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'java/com/android/dialer') diff --git a/java/com/android/dialer/app/calllog/CallLogListItemViewHolder.java b/java/com/android/dialer/app/calllog/CallLogListItemViewHolder.java index babf1f8ca..2baf117dc 100644 --- a/java/com/android/dialer/app/calllog/CallLogListItemViewHolder.java +++ b/java/com/android/dialer/app/calllog/CallLogListItemViewHolder.java @@ -740,7 +740,7 @@ public final class CallLogListItemViewHolder extends RecyclerView.ViewHolder } boolean isBlockedOrSpam = blockId != null || (isSpamFeatureEnabled && isSpam); - + if (!isBlockedOrSpam && info != null && UriUtils.isEncodedContactUri(info.lookupUri)) { createNewContactButtonView.setTag( IntentProvider.getAddContactIntentProvider( @@ -1034,8 +1034,8 @@ public final class CallLogListItemViewHolder extends RecyclerView.ViewHolder && intent.getIntExtra(TelecomManager.EXTRA_START_CALL_WITH_VIDEO_STATE, -1) == VideoProfile.STATE_BIDIRECTIONAL) { Logger.get(mContext).logImpression(DialerImpression.Type.IMS_VIDEO_REQUESTED_FROM_CALL_LOG); - } - if (intent.getDataString().contains(DuoConstants.PACKAGE_NAME)) { + } else if (intent.getDataString() != null + && intent.getDataString().contains(DuoConstants.PACKAGE_NAME)) { Logger.get(mContext).logImpression(DialerImpression.Type.DUO_CALL_LOG_SET_UP_INSTALL); } -- cgit v1.2.3 From f539f7885be23d3aff470e573c70d52db3b2a878 Mon Sep 17 00:00:00 2001 From: yueg Date: Mon, 18 Dec 2017 16:20:58 -0800 Subject: Bubble v2 logging. Log the following actions: - expand/collapse bubble - return to call - mute/unmute - switch audio route to speaker/bluetooth/wired or earpiece - end call Bug: 67605985 Test: AudioRouteSelectorActivityTest, NewReturnToCallControllerTest PiperOrigin-RevId: 179484647 Change-Id: I4e2ee34f5550382b2e51bab16ce33e9e16caa3b2 --- .../android/dialer/logging/dialer_impression.proto | 21 ++++++++++++++++++++- 1 file changed, 20 insertions(+), 1 deletion(-) (limited to 'java/com/android/dialer') diff --git a/java/com/android/dialer/logging/dialer_impression.proto b/java/com/android/dialer/logging/dialer_impression.proto index aed66251a..2d2eebf67 100644 --- a/java/com/android/dialer/logging/dialer_impression.proto +++ b/java/com/android/dialer/logging/dialer_impression.proto @@ -12,7 +12,7 @@ message DialerImpression { // Event enums to be used for Impression Logging in Dialer. // It's perfectly acceptable for this enum to be large // Values should be from 1000 to 100000. - // Next Tag: 1311 + // Next Tag: 1320 enum Type { UNKNOWN_AOSP_EVENT_TYPE = 1000; @@ -625,5 +625,24 @@ message DialerImpression { DUO_CALL_LOG_SET_UP_INSTALL = 1308; DUO_CALL_LOG_SET_UP_ACTIVATE = 1309; DUO_CALL_LOG_INVITE = 1310; + + // Bubble primary button first click to expand bubble + BUBBLE_V2_CLICK_TO_EXPAND = 1311; + // Bubble primary button second click to collapse bubble + BUBBLE_V2_CLICK_TO_COLLAPSE = 1312; + // User return to call from bubble call action menu + BUBBLE_V2_RETURN_TO_CALL = 1313; + // User muted call from bubble call action menu + BUBBLE_V2_MUTE_CALL = 1314; + // User unmuted call from bubble call action menu + BUBBLE_V2_UNMUTE_CALL = 1315; + // User choose audio route speakerphone from bubble call action menu + BUBBLE_V2_SPEAKERPHONE = 1316; + // User choose audio route wired or earpiece from bubble call action menu + BUBBLE_V2_WIRED_OR_EARPIECE = 1317; + // User choose audio route bluetooth from bubble call action menu + BUBBLE_V2_BLUETOOTH = 1318; + // User ended call from bubble call action menu + BUBBLE_V2_END_CALL = 1319; } } -- cgit v1.2.3