Disabling scenarioChromeURLInvalid |
|||||||||||
Issue descriptionIt fails after chromium roll to b65b322f74351da8b61fe35889348649b56d213d. See https://chromereviews.googleplex.com/499897013/ for the revert. The failure is that it can't find IDS_ERRORPAGES_HEADING_NOT_AVAILABLE which is "This site can’t be reached". Instead the UI show only a black line (see screenshot) To Eugene for triage.
,
Sep 2 2016
,
Sep 2 2016
Nit quite sure what happened here but everything works fine in app and test if I run locally. Eric, were you able to reproduce this bug locally? Maybe something was broken downstream and then fixed?
,
Sep 2 2016
Can not reproduce by running test locally or by running the app itself. Looks like a problem with test which only happens on bots. We should wait until Claude ports it to Earl Grey, but I'm keeping this bug as open and RBS until KIF tests are shut down.
,
Sep 5 2016
It broke when Chromium rolled. I suspect something changed on error pages.
,
Sep 6 2016
Claude does EG test failing for you on bots, I remember you started converting it last week? CCing Jackie for WebUI, but per comment #4 the error page is displayed correctly if I run the app. I suspect test problem here and if EG version works fine, then we may not even need to fix KIF.
,
Sep 6 2016
Ok, so EG tests are also failing: https://chromereviews.googleplex.com/501987013/
,
Sep 6 2016
Jackie, can you take a look. This is not reproducible locally for some reason, and it looks like the breakage came from upstream code.
,
Sep 6 2016
Thanks for taking a look Jackie.
,
Sep 8 2016
,
Oct 3 2016
I think we can land https://chromereviews.googleplex.com/501987013/ now, with both KIF and EG enabled. KIF passes on ToT. Reassigning to Claude as owner of the conversion CL.
,
Oct 4 2016
Landed.
,
Oct 28 2016
[Auto-generated comment by a script] We noticed that this issue is targeted for M-55; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-55 label, otherwise remove Merge-TBD label. Thanks.
,
Oct 31 2016
,
Jan 24 2018
|
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by bugdroid1@chromium.org
, Sep 2 2016