Reland "Refined DCHECK preventing multiple event forwarders in VA tree path"

ui/android/view_android.h

Index: ui/android/view_android.h
diff --git a/ui/android/view_android.h b/ui/android/view_android.h
index fbd3c452cd1bfc555b5e50c828b7d4f2ad255c22..0fdc5983f0ea255b2bd6487eaa5e1335558ce718 100644
--- a/ui/android/view_android.h
+++ b/ui/android/view_android.h
@@ -178,11 +178,11 @@ class UI_ANDROID_EXPORT ViewAndroid {
 
   bool has_event_forwarder() const { return !!event_forwarder_; }
 
-  // Returns true if any node of the tree along the hierarchy (view's children
-  // and parents) already has |EventForwarder| attached to it.
-  static bool ViewTreeHasEventForwarder(ViewAndroid* view);
+  // Checks if there is any event forwarder in any node up to root.
+  static bool RootPathHasEventForwarder(ViewAndroid* view);
 
-  // Returns true if any children node (or self) has |EventForwarder|.
+  // Checks if there is any event forwarder in the node paths down to
+  // each leaf of subtree.
   static bool SubtreeHasEventForwarder(ViewAndroid* view);
 
   // Returns the Java delegate for this view. This is used to delegate work