Chromium Code Reviews
DescriptionRevert of Refined DCHECK preventing multiple event forwarders in VA tree path (patchset #9 id:200001 of https://codereview.chromium.org/2861413002/ )
Reason for revert:
Primary suspect making ViewAndroidTest.ChecksMultipleEventForwarders fails consistently
https://uberchromegw.corp.google.com/i/chromium.linux/builders/Android%20Tests
Original issue's description:
> Refined DCHECK preventing multiple event forwarders in VA tree path
>
> A ViewAndroid tree path from the top to a leaf node can have only
> a single EventForwarder instance. Rewrote helper methods used for
> DCHECK that enforces the requirement to match the actual job with
> the intention.
>
> BUG=717271
>
> Review-Url: https://codereview.chromium.org/2861413002
> Cr-Commit-Position: refs/heads/master@{#470814}
> Committed: https://chromium.googlesource.com/chromium/src/+/1a116ecfdf829702b653ded84a0e1cdb8bdfd7d6
TBR=boliu@chromium.org,jinsukkim@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=717271
Review-Url: https://codereview.chromium.org/2880583002
Cr-Commit-Position: refs/heads/master@{#470938}
Committed: https://chromium.googlesource.com/chromium/src/+/38ac4c9a36f237bcddf1ae1e723a7ee00c841f64
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|
|||||||||||||||||||||||||||||||||||||