New issue
Advanced search Search tips

Issue 730192 link

Starred by 0 users
Status: Available
Owner: ----
Cc:
eugene...@chromium.org
kkhorimoto@chromium.org
ios-bugs@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Task

Blocking:
issue 676129



Sign in to add a comment

Have only one path from navigationManager for reload

Project Member Reported by mrefaat@chromium.org, Jun 6 2017 
Have only one path from navigationManager for reloads.

Right now, when the user focus on Omni-box and press go again on the keyboard this go through loadURLWithParams instead of calling reload, although the transition type is reload.

Comment 1 by pkl@chromium.org, Jun 7 2017

Status: Assigned (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 8 2017 

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

commit a84d5a044c5401028b18d84c738ac1f7fdfc2c2c
Author: mrefaat <mrefaat@chromium.org>
Date: Thu Jun 08 17:13:29 2017

Call reload when user initiate reload using the omnibox

BUG=730192

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

[modify] https://crrev.com/a84d5a044c5401028b18d84c738ac1f7fdfc2c2c/ios/chrome/browser/ui/browser_view_controller.mm

Comment 3 by eugene...@chromium.org, Jul 10 2017

Status: Started (was: Assigned)

Comment 4 by eugene...@chromium.org, Jul 11 2017

Labels: Proj-Not-WKBackForwardList

Comment 5 by eugene...@chromium.org, Jan 15 

Should we flip this to Available?

Comment 6 by mrefaat@google.com, Jan 15

Owner: ----
Status: Available (was: Started)
yes
►

Sign in to add a comment