Error is thrown by controller.jsm each time a URL is dragged onto the Bookmarks Toolbar

RESOLVED FIXED in Firefox 61

Status

()

P3
normal
RESOLVED FIXED
4 years ago
10 months ago

People

(Reporter: avaida, Assigned: standard8)

Tracking

({regression})

Trunk
mozilla61
regression
Points:
---

Firefox Tracking Flags

(firefox39 wontfix, firefox40 wontfix, firefox41 wontfix, firefox-esr31 wontfix, firefox-esr52 wontfix, firefox59 wontfix, firefox60 wontfix, firefox61 fixed)

Details

(Whiteboard: [Fixed by bug 1439697])

Reproducible on:
* 31.8 ESR Build 1 (20150624141335)
* 39.0 RC Build 4 (20150624153222)
* Aurora 40.0a2 (2015-06-24)
* Nightly 41.0a1 (2015-06-24)

Steps to reproduce:
1. Launch Firefox.
2. Enable the Bookmarks Toolbar.
3. Open a random website that's currently _not_ bookmarked.
4. Drag and drop its URL onto the Bookmarks Toolbar.
5. Check the Browser Console for js errors.

Expected result:
There are no issues nor errors encountered for this action.

Actual result:
controller.js:1621 throws the following error in the Browser Console:
> Tried to move an unmovable Places node, reverting to a copy operation.

Additional notes:
* This is _not_ a recent regression, as I've managed to track it back to Firefox 30.0 (20140605174243) so far. I'll follow up with a detailed regression range as soon as possible.
Regression range (m-c)
======================
Last good revision: ab137ddd3746 (2014-11-13)
First bad revision: 7f0d92595432 (2014-11-14)
Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=ab137ddd3746&tochange=7f0d92595432

Potential regressor:
a4d2e72afc42 - Tim Taubert — Bug 1083462 - Remove the bookmarks cache r=mak
Keywords: regression
Priority: -- → P3

Comment 2

a year ago
Just saw this occurring in 54.0.1.
(Assignee)

Comment 3

a year ago
Generally there's no harm in this error - it is just a warning for developers. It will, however, occur every time when dragging some text like URL test onto the toolbar as per comment 0 (and maybe the same for into the library window). There's no harm in it though.
(Assignee)

Comment 4

10 months ago
There's a fix for this in bug 1439697.
Assignee: nobody → standard8
Depends on: 1439697
(Assignee)

Comment 5

10 months ago
Fixed by bug 1439697.
Status: NEW → RESOLVED
Last Resolved: 10 months ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Whiteboard: [Fixed by bug 1439697]
Target Milestone: --- → mozilla61
status-firefox39: affected → wontfix
status-firefox40: affected → wontfix
status-firefox41: affected → wontfix
status-firefox59: --- → wontfix
status-firefox60: --- → wontfix
status-firefox-esr31: affected → wontfix
status-firefox-esr52: --- → wontfix
You need to log in before you can comment on or make changes to this bug.