Closed Bug 1177382 Opened 4 years ago Closed 2 years ago

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

Categories

(Toolkit :: Places, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox39 --- wontfix
firefox40 --- wontfix
firefox41 --- wontfix
firefox-esr31 --- wontfix
firefox-esr52 --- wontfix
firefox59 --- wontfix
firefox60 --- wontfix
firefox61 --- fixed

People

(Reporter: avaida, Assigned: standard8)

References

Details

(Keywords: regression, 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
Just saw this occurring in 54.0.1.
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.
There's a fix for this in bug 1439697.
Assignee: nobody → standard8
Depends on: 1439697
Fixed by bug 1439697.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Whiteboard: [Fixed by bug 1439697]
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.