Omnibox should show the URL of visible item instead of last committed item |
||||||||
Issue descriptionChrome Version: (copy from chrome://version) OS: Android 7.1.1; Nexus 6P Build/N4F26J What steps will reproduce the problem? (1) Go to www.amazon.com (2) Go to www.html-kit.com/tools/cookietester What is the expected result? Omnibox should show www.html-kit.com/tools/cookietester because this is a user initiated navigation. What happens instead? Omnibox shows www.amazon.com
,
Mar 9 2017
,
Mar 14 2017
tedchoc@, how should I generally triage Android N bugs? Is there an appropriate hotlist?
,
Mar 15 2017
Is this only applicable to N? Based on the steps, this sounds like it might apply to all android versions. We use the same logic as desktop for the text to be shown: https://cs.chromium.org/chromium/src/chrome/browser/ui/toolbar/chrome_toolbar_model_delegate.cc?l=47 Via the Android shim: https://cs.chromium.org/chromium/src/chrome/browser/ui/android/toolbar/toolbar_model_android.h?ssfr=1&l=22 DidAccessInitialDocument triggers a NavigationStateChanged(INVALIDATE_TYPE_URL) https://cs.chromium.org/chromium/src/content/browser/web_contents/web_contents_impl.cc?type=cs&q=DidAccessInitialDocument&l=4567 Which should be getting up to java: https://cs.chromium.org/chromium/src/chrome/android/java/src/org/chromium/chrome/browser/tab/TabWebContentsDelegateAndroid.java?q=NavigationStateChanged+file:android&dr=CSs&l=239 ToolbarManager: https://cs.chromium.org/chromium/src/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java?dr=CSs&q=ToolbarManager.java&l=324 LocationBar: https://cs.chromium.org/chromium/src/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java?dr=CSs&q=LocationBarLayout.java&l=2222 So to me, it looks like we are using the same logic as desktop and listening for the right signal.
,
Mar 15 2017
Well, I cannot reproduce this on my Android L device on Chrome 56 or on my desktop. reporter: what version of Chrome are you running?
,
Mar 15 2017
56.0.2924.87 (Official Build) (32 bit), and I just verified that I can still reproduce it only Nexus.
,
Mar 17 2017
askatte@, can you try to repro? Thanks.
,
Mar 28 2017
Not able to repro this issue on Nexus 6P/ N4F26J / 56.0.2924.87 (Official Build) (32 bit). Followed steps from #0.
,
May 17 2017
Lowering priority. Does not appear to be reproducible, much less widespread.
,
Jun 26 2017
liaoyuke@, can you reproduce this on a current Chrome release?
,
Oct 18 2017
@liaoyuke -- Could you please respond to Comment# 10 and update the issue which would help us triage the issue further. Thanks in Advance.
,
Dec 13 2017
No replies from the reporter (liaoyuke@); closing as not reproducible. |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by creis@chromium.org
, Mar 9 2017