Add null check before notifying CUJ events

There is a potential NPE since we didn't have null check before sending
broadcast, so add null cehck.

Bug: 205211901
Test: Manually
Change-Id: Icc60d2c87941373943ec627f669598dca684c09d
This commit is contained in:
Wu Ahan
2021-11-10 06:49:03 +00:00
parent e8e382c03f
commit eec2d1237d

View File

@@ -377,7 +377,13 @@ public class InteractionJankMonitor {
// Notify the receivers if necessary.
if (session.shouldNotify()) {
notifyEvents(context, action, session);
if (context != null) {
notifyEvents(context, action, session);
} else {
throw new IllegalArgumentException(
"Can't notify cuj events due to lack of context: cuj="
+ session.getName() + ", action=" + action);
}
}
}
@@ -739,7 +745,8 @@ public class InteractionJankMonitor {
* @return builder
*/
public static Builder withView(@CujType int cuj, @NonNull View view) {
return new Builder(cuj).setView(view);
return new Builder(cuj).setView(view)
.setContext(view.getContext());
}
private Builder(@CujType int cuj) {