Refined DCHECK preventing multiple event forwarders in VA tree path

ui/android/view_android_unittests.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/android/event_forwarder.h"
 #include "ui/android/view_android.h"
 #include "ui/android/view_client.h"
 #include "ui/events/android/motion_event_android.h"
 
@@ skipping 148 matching lines @@
   client1_.SetHandleEvent(false);
   GenerateTouchEventAt(40, 60);
   EXPECT_TRUE(client1_.EventCalled());
   ExpectHit(client2_);
 
   GenerateTouchEventAt(100, 70);
   EXPECT_TRUE(client1_.EventCalled());
   ExpectHit(client3_);
 }
 
+TEST(ViewAndroidTest, ChecksMultipleEventForwarders) {
+  ViewAndroid parent;
+  ViewAndroid child;
+  parent.GetEventForwarder();
+  child.GetEventForwarder();
+  EXPECT_DEATH(parent.AddChild(&child), "Check failed:");
+
+  ViewAndroid window;
+  ViewAndroid wcv1, wcv2;
+  ViewAndroid rwhv1a, rwhv1b, rwhv2;
+  wcv1.GetEventForwarder();
+  wcv1.GetEventForwarder();
+
+  window.AddChild(&wcv1);
+  wcv1.AddChild(&rwhv1a);
+  wcv1.AddChild(&rwhv1b);
+  wcv2.AddChild(&rwhv2);
+
+  // window can add wcv2 since there's only one event forwarder at wcv2
+  // in the path window - wcv2 - rwvh2
+  window.AddChild(&wcv2);
+
+  // Both wcv1 & rwhv1 have event forwarders in the path window - wcv1 - rwhv1a.
+  EXPECT_DEATH(rwhv1a.GetEventForwarder(), "Check failed:");
+}
+
 }  // namespace ui