Issue metadata
Sign in to add a comment
|
Regression: In 'Browser sample' app cursor is not seen on search engine even if focus ring is there.
Reported by
rk...@etouch.net,
Sep 28 2016
|
||||||||||||||||||||||
Issue descriptionChrome Version: 54.0.2840.41 Revision aeb8895aa4becddb1526af10d1e67495713b6c1a-refs/branch-heads/2840@{#557}(32/64 bit) OS: Windows(7,8,10), Mac(10.10.5,10.11.4) Pre-condition: Add 'Browser sample' third party extension from 'Load unpacked extension'. What steps will reproduce the problem? (1) Launch chrome, launch 'Browser sample' app. (2) Right click on omnibox and select Reload app option and observe. Actual: Cursor is not seen on search engine even if focus ring is seen on it. Expected: Cursor should seen on search engine or focus ring should not seen on it. This is a regression issue, broken in 'M-53', will soon update the other info:
,
Sep 28 2016
1. I can't see what is wrong with the actual result. There isn't a cursor at all in both screencasts, is there? 2. I believe the bisect result is incorrect. Mine has nothing to do with this, and all others aren't relevant, either. Please retry bisection.
,
Sep 28 2016
,
Sep 28 2016
With respect to comment 2: Rebisected for the above issue and below is bisect info: Narrow Bisect: https://chromium.googlesource.com/chromium/src/+log/61c5a5d0d32c6b995426f802a560889242968665..a6291c3ea84f371d10c0d10e66ee6d11458c9d42?pretty=fuller&n=100 Suspecting: r395325 @rob.buis: Please help me to reassign this issue, if your change is not cause for it.
,
Sep 28 2016
My change is not changing any code, only layout tests.
,
Sep 30 2016
With respect to comment 5: Re-bisected for the above issue, and getting same bisect range as mentioned in comment 4: Suspecting: r395324 @rob: Please help me to reassign this issue, if your change is not cause for it.
,
Sep 30 2016
My change only added assertions to unrelated code, try again. FWIW, I followed the steps from the video in 55.0.2877.0 on Linux and did not observe the blue outline. For testing, I downloaded the source of https://chrome.google.com/webstore/detail/browser-sample/edggnmnajhcbhlnpjnogkjpghaikidaa (also available at https://github.com/GoogleChrome/chrome-app-samples/tree/master/samples/webview-samples/browser).
,
Jan 12 2017
The assigned owner "rkote@etouch.net" is not able to receive e-mails, please re-triage. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jan 12 2017
,
Jan 13 2017
Assigning to James for triage.
,
Jan 24 2017
So I just tried this using ToT Chromium on Mac (58.0.2992.0 (64-bit)) and here are my observations: 0) Since this bug was filed, a lot of webview-focus related changes have occurred, and the original intent of this bug may no longer exist. 1) The repro instructions no longer apply - in particular, right clicking in the app's URL bar does not open a context menu with a "reload app" option. 2) Loading/reloading google.ca in the regular browser moves focus to the text input box, while loading/reloading the same site in Browser Sample does not. It seems to me that the two should act the same, but I could be wrong. avallee@ - Could you please triage this further and determine: 1) What is the current behaviour on Windows? 2) Is it reasonable to expect the same focus behaviour in the app as in the main browser window when loading google.ca?
,
Jan 24 2017
,
Jan 24 2017
wjmaclean@: I just checked canary on Windows. Observations: 1. The reload app option only shows for extensions that were loaded as unpacked in dev mode. Web store extension don't get the reload app option (makes sense that the dev options are only on the dev extensions). 2. The browser behaviour is what I'd expect from the browser sample. I've got a patch for browser sample which focused the webview on load (loads google home page) and on navigation from the url bar. 3. The current behaviour without cross process frames is that the content starts off not focused, there's no caret on the search page, the search box does not have the larger "active" shadow. 4. The current behaviour with cross process frames is that the content always gets focused, there is a caret and the shadow is slightly larger indicating that the box is active.
,
Jan 24 2017
Fixed browser sample to always have the web content focused on start and after navigation. https://github.com/GoogleChrome/chrome-app-samples/pull/468 No changes required to Chromium. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by rk...@etouch.net
, Sep 28 2016Owner: yutak@chromium.org
Status: Assigned (was: Unconfirmed)