GitLab

The call in SetRenderFrameHost was useless because the speculative RFH
has already been move out of place so the validation exits early. This
change moves the validation into a static method that takes 2 frames
as arguments, so that it can be used even in this case.

https://crrev.com/c/2564933/1 shows that this passes all tests even with
SkipEarlyCommitPendingForCrashedFrame.

Bug: 1146573
Change-Id: I59574bd78a93534ec7e0ae603589e82c2e87a119
Cq-Do-Not-Cancel-Tryjobs: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2564933Reviewed-by: Nasko Oskov <nasko@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Fergal Daly <fergal@chromium.org>
Auto-Submit: Fergal Daly <fergal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#834005}