1279329 - "Copy Download Link" of context menu in Library is grayed out

Status: RESOLVED DUPLICATE of bug 1282050

(Firefox :: Downloads Panel, defect, P3)

Description

[Alice0775 White]

[Tracking Requested - why for this release]: UI is broken due to regression by Bug 1117145

Build Identifier:
https://hg.mozilla.org/mozilla-central/rev/f8e3b81a79f45ef8647c98281a9a00d1ddb28b73
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:50.0) Gecko/20100101 Firefox/50.0 ID:20160609064045


This is regression since Firefox47

Reproducible : always

Steps To Reproduce:
1. Download any link (e.g, http://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-inbound-win32/latest/firefox-44.0a1.en-US.win32.zip )
2. Open Download Manager(Library)
3. Right click on the download entry
   --- observe "Copy Download Link" of context menu

(optionally)
4. Close The Download Manager and Reopen it
5. Right click on the download entry
   --- observe "Copy Download Link" of context menu

Actual Results:
"Copy Download Link" of context menu in Library is disabled(grayed out)

Expected Results:
"Copy Download Link" should be enabled(select-able)

Regression window:
https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=0a58b5b8a407f1420ce27b09554305c6b4bebfa2&tochange=3b3428c04743ca6da4472583dc809ae78cd3a5db

Regressed by: Bug 1117145

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

I am unable to reproduce this bug on Nightly 50 2016-06-09 Windows 10 64-bit. I tried with the library as well as the doorhanger. Screencast, http://screencast.com/t/9V0KbopwwFy

Comment 2

[Alice0775 White]

(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #1)
> I am unable to reproduce this bug on Nightly 50 2016-06-09 Windows 10
> 64-bit. I tried with the library as well as the doorhanger. Screencast,
> http://screencast.com/t/9V0KbopwwFy

You may need further the optional step4-5 of comment#0.
Or
Make sure Download Manager(Library) is closed before you start the download.

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

I did. The video also shows me closing the library and reopening it.

But, I did just reproduce it. You must open the download library from the download panel. If you open the download library from the History library (ctrl+shift+h and then click on Downloads) it works.

To reproduce you must open the downloads library from the downloads panel.

Comment 4

[Alice0775 White]

WORKAROUND:
Select the other tree item(Tags/All Bookmarks) in the left pane of Library.
And then, re-select the Download. the "Copy Download Link" function comes back.

Comment 5

[Jared Wein [:jaws] (please needinfo? me)]

Paolo, maybe since the binding is loading the data is setup yet? I tried to put some breakpoints in with the Browser Debugger but I couldn't get any to hit, maybe because it's a separate window?

Comment 6

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Hi Jared, it seems to be a recent regression in Fx47. But since there is a workaround (i.e the issue does not repro when you open the download panel with the shortcut key), I don't consider this severe for Fx47 release. Do you agree? Or do you believe this is something we should consider as a dot release ride-along? Please let me know.

Comment 7

[Alice0775 White]

I think this is minor problem, not need a dot release47 and beta48 too.

Comment 8

[Jared Wein [:jaws] (please needinfo? me)]

I agree with comment 7. This is a minor problem and doesn't occur in the downloads panel, which has higher visibility than the downloads library.

Comment 9

[Exodius Exorcist]

Confirmed on 47.0 final stable
While downlaoding http://image.prntscr.com/image/54d4dbc2c4094753a7a0805342dcb598.png
Canceled download, clear downlaod still grayed out http://image.prntscr.com/image/ce05a9e47b5749f9ab0f8e729b8212de.png

Comment 11

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Thanks Alice0775 White and Jared. We will try to fix this in 48+, it's a wontfix for Fx47.

Comment 12

[Liz Henry (:lizzard) (relman/hg->git project)]

Tracking for 49+ since this is a recent regression. I don't think we need to push to get it into beta though. Paolo can you take this on?

Comment 13

[:Paolo Amadini]

This will be solved by the patch in bug 1282050.

Comment 14

[Marcia Knous [:marcia]]

We will track for 49/50 in the duplicate bug.

Comment 15

[David Bolter [:davidb] (NeedInfo me for attention)]

Updating status flags to get this DUPE of triage lists.