New issue
Advanced search Search tips

Issue 825341 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Jan 2
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

browser_test FindInPageControllerTest.FindLongString is flaky on Win7 Tests (dbg)(1)

Project Member Reported by wolenetz@chromium.org, Mar 23 2018

Issue description

Cc: pkasting@chromium.org
CC+=one of the OWNERs - pardon if I chose incorrectly.
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 23 2018

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

commit 102d3c2cb0d22f0cd05fe213a389bbf725a452b0
Author: Matt Wolenetz <wolenetz@chromium.org>
Date: Fri Mar 23 20:49:54 2018

browser_test FindInPageControllerTest.FindLongString is flaky

It's flaky on Win7 Tests (dbg)(1).
This change disables it in debug Windows builds.

BUG= 825341 , 823003 
TBR=pkasting@chromium.org
NO-TRY=True

Change-Id: Ibb04812ca20081ec55cbc1bdbef01464ddb062f3
Reviewed-on: https://chromium-review.googlesource.com/978835
Reviewed-by: Matthew Wolenetz <wolenetz@chromium.org>
Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545574}
[modify] https://crrev.com/102d3c2cb0d22f0cd05fe213a389bbf725a452b0/chrome/browser/ui/find_bar/find_bar_host_browsertest.cc

Labels: Hotlist-DesktopUIChecked
Status: WontFix (was: Untriaged)
***UI Mass Triage ***

Closing, guessing this is fixed, feel free to reopen if needed. 

Status: Available (was: WontFix)
The test is still disabled, so not fixed
Labels: Hotlist-DesktopUIToolingRequired
Project Member

Comment 6 by bugdroid1@chromium.org, Jan 2

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

commit dca159597606f91cf28b847c90981e4acffafde1
Author: Peter Kasting <pkasting@chromium.org>
Date: Wed Jan 02 19:15:01 2019

Re-enable disabled tests in c/b/ui/find_bar/.

These pass on Windows.

Bug:  11761 ,  40948 ,  43070 ,  43072 ,  367701 ,  369169 ,  733286 ,  825341 
Change-Id: Ic6ebbddbdd306ff09867a1ea2c97cbf701ee526e
Reviewed-on: https://chromium-review.googlesource.com/c/1351785
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#619448}
[modify] https://crrev.com/dca159597606f91cf28b847c90981e4acffafde1/chrome/browser/ui/find_bar/find_bar_host_browsertest.cc
[modify] https://crrev.com/dca159597606f91cf28b847c90981e4acffafde1/chrome/browser/ui/find_bar/find_bar_host_interactive_uitest.cc

Status: Fixed (was: Available)

Sign in to add a comment