Issue metadata
Sign in to add a comment
|
Regression: Wrong View site info bubble is seen instead of origin bubble for 'Google Docs' page.
Reported by
rk...@etouch.net,
Nov 30 2016
|
||||||||||||||||||||||||
Issue descriptionChrome Version: 57.0.2936.0 Revision 325a49517707810d1abdfb0040e19b7abb5addbe-refs/heads/master@{#434845} OS: Windows (7,8,10), Mac (10.11.6, 10.12.1), Linux (14.04 LTS) What steps will reproduce the problem? (1) Launch chrome, navigate to chrome://apps and click on 'Google Docs' (2) Click on View site info bubble immediately and observe. Actual: Wrong View site info bubble is seen (i.e. You are viewing extension page) instead of origin bubble. Expected: Origin bubble should seen. This is a regression issue, broken in 'M-56', will soon update the other info: Good build: 56.0.2902.0 Bad Build: 56.0.2906.0
,
Nov 30 2016
rdevlin@ The suspected CL is a reland of your original CL here: https://codereview.chromium.org/2452773002/ Could you please have a look.
,
Dec 2 2016
This actually doesn't look wrong to me. If you look closely at both videos, in the "actual" one, the user clicks the site info bubble while the page is pointing to an extension page. In the "expected" one, the extension page url flashes but the user doesn't click until redirected to docs.google.com (and then ultimately to the sign in page). I think this is WAI, but +felt@ and creis@ for their opinions.
,
Dec 2 2016
Comment 3: I agree. In both videos, the bubble displays information about the URL that was present at the time of the click. That's how it should behave. However, I do wonder if we should be dismissing the bubble if a new commit happens while it's showing. That would avoid showing stale info that no longer corresponds to the current page. felt@ is OOO. emilyschechter@, do you have thoughts about that or know who to loop in?
,
Dec 2 2016
Adding +meacer and +lgarron to triage. Dismissing or updating the bubble sounds reasonable, I think.
,
Dec 2 2016
I agree with #3 and #4 that the initial contents of the bubble look correct in both the actual and expected cases. Current stable version also exhibits same behavior so there doesn't seem to be a regression here. Filed bug 670787 to hide or update the bubble on a new commit and wontfixing this. (msrchandra, rkote: Let us know if we are missing anything)
,
Dec 6 2016
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by msrchandra@chromium.org
, Nov 30 2016Owner: xidac...@chromium.org
Status: Assigned (was: Unconfirmed)