summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Philip Cuadra <philipcuadra@google.com> 2021-06-07 17:21:37 -0700
committer Philip Cuadra <philipcuadra@google.com> 2021-06-08 13:34:21 -0700
commitef671105e18c64f6ae3ee4e7cad75c641ab95a74 (patch)
treea7c43e246cf853ee59062afad084e9eb26b68f0b
parentd8be4517b3da4ab236fd04dd9b85b528f38cf5b8 (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.java16
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;