Regression:Blank page appears after dragging url from omnibox on page.
Reported by
vku...@etouch.net,
Jun 13 2016
|
||||||||||
Issue descriptionChrome Version: 53.0.2766.0 (Official Build) e40502b71c9bd4f548118550952afd5d6a158bc4-refs/heads/master@{#399363} 32/64-bit. OS:Mac (10.10.5 , 10.11.4),Mac Retina (10.11.4) What steps will reproduce the problem? 1.Launch chrome and navigate to chrome://settings or any webpage. 2.Right click and select 'view page source' from context menu. 3.After loading the page drag url from omnibox onto the page and observe. Actual: Blank page appears after dragging url of 'view page source' on page. Expected: Blank page should not be seen after dragging url of 'view page source' on page. This is a regression issue broken in 'M53' and will soon update other info.
,
Jun 13 2016
Adding RB label as this is a recent regression.
,
Jun 13 2016
Broken by r397505: We blocked page initiated navigations to view-source urls. I didn't know that dragging & dropping URLs into the page cause a page-initiated navigation, I'd expect them to be browser initiated. Charlie, any thoughts? Do you think this is a wontfix?
,
Jun 13 2016
This shouldn't be WontFix. I think ideally this should be considered browser-initiated.
,
Jun 13 2016
That's what I was thinking, but I'm not sure if there is an argument against it.
,
Jun 20 2016
creis@ : Could you please take a look into this as issue still persists on the latest canary 53.0.2773.0
,
Jun 21 2016
How is this a P1 stable blocker? It only affects dragging view-source URLs into the content area. That seems very uncommon to me, so I'd suggest lowering the priority to P2 and removing the blocking label. I'm not sure if it's possible to treat dragged URLs as browser-initiated navigations or not. dcheng@, do you know how dragged navigations work? I imagine the renderer is the one that decides to translate the drag into a navigation (much like link clicks and other renderer-initiated navigations). It seems problematic to say that the renderer can't do navigations to view-source URLs while allowing it to do drag navigations to them. Wouldn't that make the protection in r397505 meaningless, such that a compromised renderer could always just say it's a drag operation?
,
Jun 21 2016
Clearly we can't let the renderer make that determination. At the risk of speaking out of my ignorance, I think the way this should work is that the renderer should decide whether something in the page content is going to consume the drag contents (e.g. for when you're dragging an image onto Google Image Search). If nothing does, the browser is free to do a browser-initiated navigation of the renderer to the desired URL.
,
Jun 21 2016
It's possible to make this work. Basically, in https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/page/DragController.cpp?rcl=0&l=274, we'd want to have it call back out to the browser to navigate on its behalf. In theory, pretty simple...
,
Jun 21 2016
(The browser will have to save a copy of the URL, so the call out to the embedder would just be an IPC telling it to take a navigation with the drag data it already has)
,
Jun 29 2016
based on c#7, reducing the priority and removing blocker label. Please add if required.
,
Jul 5 2016
Moving this nonessential bug to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 3 2016
Note:Above issue is still reproducible on latest canary version i.e 55.0.2878.0 (Official Build)
,
Oct 21 2016
Issue 622589 has been merged into this issue.
,
Jun 16 2017
Still reproduces (dragging a view-source: URL to the content area causes a navigation to about:blank). meacer@, are you still planning to fix this? Removing DataTransfer and Options components, adding Navigation component.
,
Jun 16 2017
> meacer@, are you still planning to fix this? It's low priority, so yes but not anytime soon. If anyone else is interested, please feel free to take it. |
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by vku...@etouch.net
, Jun 13 2016Labels: hasbisect OS-Linux OS-Windows
Owner: mea...@chromium.org
Status: Assigned (was: Unconfirmed)
866 KB
866 KB Download
1.0 MB
1.0 MB Download