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..a41e415bf1cd111062239ecd4b6030cecf235e79 100644
--- a/ui/android/view_android.h
+++ b/ui/android/view_android.h
@@ -178,12 +178,14 @@ 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);
-
-  // Returns true if any children node (or self) has |EventForwarder|.
-  static bool SubtreeHasEventForwarder(ViewAndroid* view);
+  // Checks if there is any path from leaf to top that has more than one event
+  // forwarder if |tree| is added to |node| as a child. |has_forwarder| is true
+  // if |node| has (or will have) an event forwarder.
+  static bool HasMultipleEventForwarder(ViewAndroid* node,
+                                        bool has_forwarder,
+                                        ViewAndroid* tree);
+  static bool HasMultipleEventForwarderInAnyPath(ViewAndroid* node,
+                                                 bool forwarder_found);
 
   // Returns the Java delegate for this view. This is used to delegate work
   // up to the embedding view (or the embedder that can deal with the