Commit effa7da6 authored by shaktisahu's avatar shaktisahu Committed by Commit bot

Fixed disabled status of Find-in-page previous/next arrows

Currently the previous/next arrows are not disabled when there is no
text match. Fixed this.

BUG=509857

Review-Url: https://codereview.chromium.org/2584983002
Cr-Commit-Position: refs/heads/master@{#439532}
parent 3b3d36ba
......@@ -235,7 +235,6 @@ public class FindToolbar extends LinearLayout
if (mFindInPageBridge == null) return;
mAccessibilityDidActivateResult = false;
setPrevNextEnabled(s.length() > 0);
if (mSettingFindTextProgrammatically) return;
......@@ -430,6 +429,8 @@ public class FindToolbar extends LinearLayout
result.numberOfMatches);
setStatus(text, result.numberOfMatches == 0);
setPrevNextEnabled(result.numberOfMatches > 0);
// The accessible version will be something like "Result 1 of 9".
String accessibleText = getAccessibleStatusText(
Math.max(result.activeMatchOrdinal, 0),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment