+ios sheriff

This is more general than just chrome_integration_egtests. Lot of tests are failing with timeout. More probably an issue with bots?

It seems to time out on SigninInteractionControllerTestCase/testSignInOneUser - not sure though. jlebel@ can you ptal?

cc'ing jbudorick and sergey in case this is related to other timeouts.

Looks like timeouts on https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/ios-slimnav?limit=100 started occuring in this precise range https://chromium.googlesource.com/chromium/src/+log/638e15167f366f1808c9122577ab881d18b25331..b485a9cce7a08fad618603f6759cc3167b540b8f?pretty=fuller&n=10000

Only ios/ CL in there is https://chromium-review.googlesource.com/c/chromium/src/+/1419497; I'll try a revert.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2a974e5ae5d7df2535b2923999f48a075d5394a8

commit 2a974e5ae5d7df2535b2923999f48a075d5394a8
Author: Gabriel Charette <gab@chromium.org>
Date: Fri Jan 18 17:20:53 2019

Revert "[ios] Call LoadIfNecessary even when displaying the NTP."

This reverts commit ddf184b056b4c7ddd8e1bd697c17924b6b50ed58.

Reason for revert: suspect for crbug.com/916159
(only ios CL in range : https://chromium.googlesource.com/chromium/src/+log/638e15167f366f1808c9122577ab881d18b25331..b485a9cce7a08fad618603f6759cc3167b540b8f?pretty=fuller&n=10000)

Original change's description:
> [ios] Call LoadIfNecessary even when displaying the NTP.
> 
> WKBasedNavigationManagerImpl::Restore will never actually begin loading
> if the current page is an NTP.  Call LoadIfNecessary to begin restore
> process.
> 
> Change-Id: Iebcfee2680ca665ebdb15883772fc042366eac58
> Reviewed-on: https://chromium-review.googlesource.com/c/1419497
> Commit-Queue: Justin Cohen <justincohen@chromium.org>
> Reviewed-by: Danyao Wang <danyao@chromium.org>
> Reviewed-by: Eugene But <eugenebut@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#623950}

TBR=justincohen@chromium.org,eugenebut@chromium.org,danyao@chromium.org

Bug: 916159
Change-Id: I7fc9766b397c97a774354d9cb28d3b38be4e5594
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/1422004
Reviewed-by: Gabriel Charette <gab@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#624175}
[modify] https://crrev.com/2a974e5ae5d7df2535b2923999f48a075d5394a8/ios/chrome/browser/ui/browser_view_controller.mm

Revert in r624175 fixed this.

There's definitely something going on with  ddf184b056b4c7d and 18cfd8ad35095 wrt testSignInSwitchManagedAccount, but none of this is related to ios_chrome_ui_egtests timetouts.  

Marking this bug as available.