Issue 676045 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	shaktisahu@chromium.org
Closed:	Dec 2016
Components:	Tests>Flaky UI>Browser>FindInPage
EstimatedDays:	----
NextAction:	----
OS:	Android
Pri:	1
Type:	Bug

FindTest#testBackKeyDismissesFind failing on tablets

Project Member Reported by dfalcant...@chromium.org, Dec 20 2016

Issue description

Guessing it's related to a CL that landed yesterday.  Close enough to the timeframe that a revert might be warranted, if that's the cause.

Possible CL:
https://codereview.chromium.org/2581923003

Marshmallow bot failure with CL in range:
https://build.chromium.org/p/chromium.android/builders/Marshmallow%20Tablet%20Tester/builds/6810

Test failures:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=org.chromium.chrome.browser.widget.findinpage.FindTest%23testBackKeyDismissesFind&testType=chrome_public_test_apk

Project Member

Comment 1 by bugdroid1@chromium.org, Dec 21 2016

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

commit 29e3616a6cf88269d143d302b3229ece42666065
Author: shaktisahu <shaktisahu@chromium.org>
Date: Wed Dec 21 00:12:37 2016

Fixed and added Find-in-page tests

Fixed a test for Find in page. Also added two new tests to verify
behavior on fullscreen mode and prev/next button functionality in
case of no match.

BUG= 676045 

Review-Url: https://codereview.chromium.org/2595713002
Cr-Commit-Position: refs/heads/master@{#439945}

[modify] https://crrev.com/29e3616a6cf88269d143d302b3229ece42666065/chrome/android/javatests/src/org/chromium/chrome/browser/widget/findinpage/FindTest.java

Comment 2 by shaktisahu@chromium.org, Dec 21 2016

Status: Fixed (was: Assigned)

This test was broken due to CL: https://codereview.chromium.org/2584983002/
Fixed the test.

►

Issue 676045 link

Issue metadata

FindTest#testBackKeyDismissesFind failing on tablets

Issue description

Comment 1 by bugdroid1@chromium.org, Dec 21 2016

Comment 1 Deleted

Comment 2 by shaktisahu@chromium.org, Dec 21 2016

Comment 2 Deleted

Sign in to add a comment