New issue
Advanced search Search tips

Issue 710506 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Mar 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug

Blocking:
issue 671916
issue 673589



Sign in to add a comment

MacViews: Find bar could lose focus after switching tabs

Reported by mbl...@yandex-team.ru, Apr 11 2017

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.110 YaBrowser/17.4.0.1854 (beta) Yowser/2.5 Safari/537.36

Steps to reproduce the problem:
1. Open a browser window with two tabs;
2. Initiate a Find command on one of the tabs;
3. Switch to the other tab, focus the tab contents;
4. Switch back: find bar will not be focused, focus will be either on bookmarkbar or on some other thing if bookmark bar is hidden.

What is the expected behavior?
Focus will be still on the find bar.

What went wrong?
Focus was restored incorrectly.

Did this work before? No 

Chrome version: 57.0.2987.110  Channel: n/a
OS Version: OS X 10.12.3
Flash Version: 

This works fine on Windows and in Cocoa.
 
screencast 2017-04-11 19-59-25.mp4
4.4 MB View Download
There's a FindInPageTest.FocusRestoreOnTabSwitch, but it currently fails on MacViews.

Comment 2 by meh...@chromium.org, Apr 11 2017

Components: UI>Browser>FindInPage
Labels: Proj-MacViews

Comment 3 by tapted@chromium.org, Apr 12 2017

Blocking: 671916 673589

Comment 4 by tapted@chromium.org, Apr 12 2017

Status: Available (was: Unconfirmed)
Labels: MacViews-Phase-4
Labels: -MacViews-Phase-4 MacViews-Dialogs
Labels: -MacViews-Dialogs MacViews-Browser
The find bar itself is staying Cocoa for now, so this is a MacViews-Browser bug.
Status: Fixed (was: Available)
MacViews triage: this now behaves correctly (the find bar has focus when switching back).
I tried this in master's build (mac_views_browser = true) and it still is reproduced.
Version 68.0.3397.0 (Developer Build) (64-bit)
8e604d185f1e57482b98d60a8d54511e959344f3


Sign in to add a comment