Downloads created outside the "legacy" download saver have no referrer info which breaks the "go to download page" context menu
Categories
(Firefox :: Downloads Panel, defect, P3)
Tracking
()
People
(Reporter: Gijs, Assigned: aminomancer)
References
Details
(Whiteboard: [fidefe-Outreachy2021])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Example steps:
- open https://xkcd.com/
- right click the comic, click "inspect" to open the inspector
- right click the node in the inspector, click "screenshot node
- open the downloads panel, right click the downloads entry
- click "go to download page.
ER:
it goes somewhere
AR:
nothing happens.
We should either disable the menu entry or fix the createDownload
API to require referrer info and update all the consumers to provide some.
Updated•3 years ago
|
Reporter | ||
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 1•3 years ago
|
||
The "Go To Download Page" menuitem in the downloads panel context menu
currently is always visible, even if the download is missing referrer
info. A download source should ideally always have referrer info,
but it's worth having a fallback so user doesn't see failures.
This patch also adds hiding for the "Copy Location" menuitem in the
unlikely event that a download is lacking a source or the source is
somehow lacking a URL. It implements a test to confirm hiding works.
I checked other downloads panel context menu tests to make sure they
aren't broken by the menuitems potentially being hidden.
This shouldn't close Bug 1723712 since we should ensure all downloads
have referrer info. This is just a stopgap in the meantime.
Updated•3 years ago
|
Updated•3 years ago
|
Comment 3•3 years ago
|
||
bugherder |
Updated•2 years ago
|
Assignee | ||
Comment 4•2 years ago
|
||
Did we just keep this open so we could add referrer info for all the download creators?
Comment 5•2 years ago
|
||
it would be better to file a new bug anyway, than to keep bugs with landed patches open.
Updated•2 years ago
|
Comment 6•2 years ago
|
||
I added leave-open because I read the last paragraph of comment #1.
Assignee | ||
Comment 7•2 years ago
|
||
Thanks, I missed that 😅 we can work on the referrer info issue in bug 1727292
Description
•