New issue
Advanced search Search tips

Issue 887962 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 22
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug
Flaky-Test: PreviewsLitePageServerBrowserTest.LitePagePreviewsResponse



Sign in to add a comment

PreviewsLitePageServerBrowserTest.LitePagePreviewsResponse is flaky

Project Member Reported by Findit, Sep 21

Issue description

Cc: anthonyvd@chromium.org
Owner: robertogden@chromium.org
Status: Assigned (was: Untriaged)
+robertogden@ who changed that test file recently. Can you please help debug or triage this flaky test? Thanks!
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 21

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

commit 68737a824338180b392d47bddd96450914b829f3
Author: Robert Ogden <robertogden@chromium.org>
Date: Fri Sep 21 21:24:10 2018

Fix flaky Lite Page Browser test

Removes the random query param hack and prevents the timeout logic from
running in all tests.

Bug:  887962 , 885252
Change-Id: Icc001d2e6d584228027ae56a12813c02dcaf0b42
Reviewed-on: https://chromium-review.googlesource.com/1239393
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Commit-Queue: Robert Ogden <robertogden@chromium.org>
Cr-Commit-Position: refs/heads/master@{#593338}
[modify] https://crrev.com/68737a824338180b392d47bddd96450914b829f3/chrome/browser/previews/previews_lite_page_browsertest.cc
[modify] https://crrev.com/68737a824338180b392d47bddd96450914b829f3/chrome/browser/previews/previews_lite_page_navigation_throttle.cc

Status: Fixed (was: Assigned)

Sign in to add a comment