New issue
Advanced search Search tips

Issue 821807 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 1
Type: Task
Q2

Blocking:
issue 828002



Sign in to add a comment

Location Bar: exit fullscreen on first tap when scrolled away, only focus on second tap

Project Member Reported by stkhapugin@chromium.org, Mar 14 2018

Issue description

When tapped, the defocused location bar focuses the omnibox.
Except when it's scrolled away in fullscreen - in this case, it should quit fullscreen and not focused.
 
Labels: -Q1 Q2

Comment 2 by cmasso@google.com, Apr 5 2018

Labels: Pri-1
Status: Started (was: Assigned)
Status: Assigned (was: Started)
Cc: kkhorimoto@chromium.org
Thanks to Kurt, there's now a simple API to do this: http://crrev.com/c/1009068
Labels: Proj-UIRefresh
Blocking: 828002
Cc: justincohen@chromium.org
Status: Started (was: Assigned)
I ran into  crbug.com/856220  while implementing this, but otherwise CL 1113749 should do the trick. 
Status: Fixed (was: Started)
Project Member

Comment 12 by bugdroid1@chromium.org, Jun 26 2018

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

commit ec9769001347865fca0f2dbda5c12c8a7d7b7875
Author: stkhapugin@chromium.org <stkhapugin@chromium.org>
Date: Tue Jun 26 09:14:11 2018

[UI Refresh] Exit fullscreen when tapping the fullscreen steady locbar.

When the steady location bar is tapped when in scrolled-away state, it
now exits fullscreen first; only the second tap focuses the omnibox.

Bug:  821807 
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: Iadaed09fa545d4335872f4f5990d6201a2351782
Reviewed-on: https://chromium-review.googlesource.com/1113749
Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570353}
[modify] https://crrev.com/ec9769001347865fca0f2dbda5c12c8a7d7b7875/ios/chrome/browser/ui/location_bar/location_bar_coordinator.mm

Sign in to add a comment