commit c950a9b832540d643e2309df6b9413f0b07444ac
author weijiaxu <weijiaxu@google.com> Mon Nov 06 16:39:04 2017 -0800
committer Zachary Heidepriem <zachh@google.com> Sat Nov 11 12:15:57 2017 -0800
tree 56df4e9d64e25c9c5de523a7610c4b009db3db32
parent f15085f9ff240b5f0bc33da02a8d8cb239458de3

Fix previous usage of LoggingBinding with Logger class.

Test: In dialer lab.
PiperOrigin-RevId: 174781953
Change-Id: I987c0d191a7c5613debb857bab9438d0c1aa20a7

java/com/android/incallui/InCallActivity.java

diff --git a/java/com/android/incallui/InCallActivity.java b/java/com/android/incallui/InCallActivity.java
index 34ffdf5..cdab6b4 100644
--- a/java/com/android/incallui/InCallActivity.java
+++ b/java/com/android/incallui/InCallActivity.java
@@ -42,7 +42,6 @@
 import com.android.dialer.logging.DialerImpression;
 import com.android.dialer.logging.Logger;
 import com.android.dialer.logging.LoggingBindings;
-import com.android.dialer.logging.LoggingBindingsFactory;
 import com.android.dialer.logging.ScreenEvent;
 import com.android.incallui.answer.bindings.AnswerBindings;
 import com.android.incallui.answer.protocol.AnswerScreen;
@@ -148,14 +147,10 @@
     pseudoBlackScreenOverlay = findViewById(R.id.psuedo_black_screen_overlay);
     sendBroadcast(CallPendingActivity.getFinishBroadcast());
     Trace.endSection();
-    if (getApplicationContext() instanceof LoggingBindingsFactory) {
-      LoggingBindings loggingBindings =
-          ((LoggingBindingsFactory) getApplicationContext()).newLoggingBindings();
-      loggingBindings.logStopLatencyTimer(
-          LoggingBindings.ON_CALL_ADDED_TO_ON_INCALL_UI_SHOWN_INCOMING);
-      loggingBindings.logStopLatencyTimer(
-          LoggingBindings.ON_CALL_ADDED_TO_ON_INCALL_UI_SHOWN_OUTGOING);
-    }
+    Logger.get(this)
+        .logStopLatencyTimer(LoggingBindings.ON_CALL_ADDED_TO_ON_INCALL_UI_SHOWN_INCOMING);
+    Logger.get(this)
+        .logStopLatencyTimer(LoggingBindings.ON_CALL_ADDED_TO_ON_INCALL_UI_SHOWN_OUTGOING);
   }
 
   @Override
@@ -199,13 +194,9 @@
     Trace.endSection();
     // add 1 sec delay to get memory snapshot so that dialer wont react slowly on resume.
     ThreadUtil.postDelayedOnUiThread(
-        () -> {
-          if (getApplicationContext() instanceof LoggingBindingsFactory) {
-            ((LoggingBindingsFactory) getApplicationContext())
-                .newLoggingBindings()
-                .logRecordMemory(LoggingBindings.INCALL_ACTIVITY_ON_RESUME_MEMORY_EVENT_NAME);
-          }
-        },
+        () ->
+            Logger.get(this)
+                .logRecordMemory(LoggingBindings.INCALL_ACTIVITY_ON_RESUME_MEMORY_EVENT_NAME),
         1000);
   }