diff options
| author | 2021-06-07 17:21:37 -0700 | |
|---|---|---|
| committer | 2021-06-08 13:34:21 -0700 | |
| commit | ef671105e18c64f6ae3ee4e7cad75c641ab95a74 (patch) | |
| tree | a7c43e246cf853ee59062afad084e9eb26b68f0b | |
| parent | d8be4517b3da4ab236fd04dd9b85b528f38cf5b8 (diff) | |
Remove tracing for notifyContentCapture
These traces get expensive in aggregate due to an increased number of
notifyContentCapture events. Remove them.
Bug 188836620
Test: Confirmed no notifyContentCapture trace points in systrace
Change-Id: I5d77bd9048b2e8cd267187f866dc095792a3f455
| -rw-r--r-- | core/java/android/view/View.java | 16 |
1 files changed, 1 insertions, 15 deletions
diff --git a/core/java/android/view/View.java b/core/java/android/view/View.java index 8138b3d30ddf..6d861dd00e9c 100644 --- a/core/java/android/view/View.java +++ b/core/java/android/view/View.java @@ -9808,23 +9808,9 @@ public class View implements Drawable.Callback, KeyEvent.Callback, */ private void notifyAppearedOrDisappearedForContentCaptureIfNeeded(boolean appeared) { AttachInfo ai = mAttachInfo; - // Skip it while the view is being laided out for the first time + // Skip it while the view is being laid out for the first time if (ai != null && !ai.mReadyForContentCaptureUpdates) return; - if (Trace.isTagEnabled(Trace.TRACE_TAG_VIEW)) { - Trace.traceBegin(Trace.TRACE_TAG_VIEW, - "notifyContentCapture(" + appeared + ") for " + getClass().getSimpleName()); - } - try { - notifyAppearedOrDisappearedForContentCaptureIfNeededNoTrace(appeared); - } finally { - Trace.traceEnd(Trace.TRACE_TAG_VIEW); - } - } - - private void notifyAppearedOrDisappearedForContentCaptureIfNeededNoTrace(boolean appeared) { - AttachInfo ai = mAttachInfo; - // First check if context has client, so it saves a service lookup when it doesn't if (mContext.getContentCaptureOptions() == null) return; |