Closed Bug 1555496 Opened 5 years ago Closed 5 years ago

Bookmarks Folder would not open when dragover a link

Categories

(Firefox :: Bookmarks & History, defect)

69 Branch
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 69
Tracking Status
firefox-esr60 --- unaffected
firefox67 --- unaffected
firefox67.0.1 --- unaffected
firefox68 --- unaffected
firefox69 --- verified

People

(Reporter: alice0775, Assigned: aswan)

References

(Regression)

Details

(4 keywords)

Attachments

(2 files)

Attached video screencast

This is regression since Nightly69.0a1

Reproducible: always (first time doesn't happen)

Steps To Reproduce:

  1. Enable Bookmarks Toolbar
  2. Create a folder in the Bookmarks Toolbar
  3. Drag a link to the folder directory(Without going through other bookmark items.)
  4. Mouse cursor shape will change to "arrow+small rectangle" from "prohibited mark"
    ---- observe, folder will open or would not open (first time doesn't happen)
  5. Cancel drag operation(press ESC key) and Click on elsewhere to close any popup
  6. Repeat from step 3

Actual Results:
The folder would not open

Expected Results:
The folder should open

Regression window:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=f287bb6c18942c75aca41d54987006951270628d&tochange=c8e9b6a81194dff2d37b4f67d23a419fd4587e49

Regressed by: Bug 1519577

Andrew Swan,
Your patch seems to cause the regression. Can you please look into this?

Flags: needinfo?(aswan)

Code here explicitly references lastElementChild inside a toolbarbutton
but it doesn't have any tests so it wasn't caught when bug 1519577 landed :(

Pushed by aswan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f6eb58afc87b
Fix bookmark toolbar drag handlers r=bgrins
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 69
Flags: needinfo?(aswan)
Flags: qe-verify+

Reproduced the issue on a Nightly from 2019-05-29.
Verified as fixed on 69.0b6 with Windows 10x64, Ubuntu 16.04 and macOS 10.12.6

Flags: qe-verify+
Status: RESOLVED → VERIFIED
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: