-
Duc Bui authored
OnWillRestoreTab() is intended to be called before any navigations of a restored WebContents but the original CL (https://chromium-review.googlesource.com/c/588168) places this function far from the creation of restored tabs. With browser-side navigation enabled, the first navigation happens before OnWillRestoreTab(), while without browser-side navigation, OnWillRestoreTab() is called before the first navigation as expected. This makes some browser tests in SessionRestorePageLoadMetricsObserver (https://crrev.com/2930013005) failed when browser-side navigation is enabled. This CL moves the call of OnWillRestoreTab() to right after the creation of a restored WebContents, before any of its navigations. Bug: 731901 Change-Id: I8d95fb14b141caffa95ada64838bde363aa10f00 Reviewed-on: https://chromium-review.googlesource.com/601545 Reviewed-by: Scott Violet <sky@chromium.org> Reviewed-by: Chris Hamilton <chrisha@chromium.org> Reviewed-by: Zhen Wang <zhenw@chromium.org> Reviewed-by: Fadi Meawad <fmeawad@chromium.org> Commit-Queue: Duc Bui <ducbui@google.com> Cr-Commit-Position: refs/heads/master@{#492992}
70c5972e