992656 - navigator.xul : "Error : DEPRECATION WARNING: Passing a callback to Livermarks methods is deprecated. Please use the returned promise instead." (Port bug 969318)

Status: RESOLVED DUPLICATE of bug 1165316

(SeaMonkey :: Bookmarks & History, defect)

Description

[Jez]

On startup of Seamonkey's browser, the following deprecation warning appears in the error console:

===
Error: DEPRECATION WARNING: Passing a callback to Livermarks methods is deprecated. Please use the returned promise instead.
You may find more details about this deprecation at: https://developer.mozilla.org/docs/Mozilla/JavaScript_code_modules/Promise.jsm
resource://gre/components/nsLivemarkService.js 404 LS_getLivemark
chrome://communicator/content/places/browserPlacesViews.js 675 PVB_containerStateChanged
chrome://communicator/content/places/browserPlacesViews.js 75 PlacesViewBase.prototype.result
chrome://communicator/content/places/browserPlacesViews.js 49 PlacesViewBase.prototype.place
chrome://communicator/content/places/browserPlacesViews.js 13 PlacesViewBase
chrome://communicator/content/places/browserPlacesViews.js 857 PlacesToolbar
chrome://communicator/content/bookmarks/browser-places.js 847 PTH_init
chrome://navigator/content/navigator.js 660 Startup
chrome://navigator/content/navigator.xul 1 onload
null 0 null

Source File: resource://gre/modules/Deprecated.jsm
Line: 79
===

Similar additional deprecation warnings appear when the sidebar is opened.  I guess the "returned promise" should be used instead.

Comment 1

[Serge Gautherie (:sgautherie)]

Agent utilisateur : Mozilla/5.0 (Windows NT 6.0; rv:33.0) Gecko/20100101 Firefox/33.0 SeaMonkey/2.30
Identifiant de compilation : 20141013232806
{
Erreur : DEPRECATION WARNING: Passing a callback to Livermarks methods is deprecated. Please use the returned promise instead.
You may find more details about this deprecation at: https://developer.mozilla.org/docs/Mozilla/JavaScript_code_modules/Promise.jsm
resource://gre/components/nsLivemarkService.js 365 LS_getLivemark
chrome://communicator/content/places/browserPlacesViews.js 659 PVB_containerStateChanged
chrome://communicator/content/places/browserPlacesViews.js 75 PlacesViewBase.prototype.result
chrome://communicator/content/places/browserPlacesViews.js 49 PlacesViewBase.prototype.place
chrome://communicator/content/places/browserPlacesViews.js 13 PlacesViewBase
chrome://communicator/content/places/browserPlacesViews.js 857 PlacesToolbar
chrome://communicator/content/bookmarks/browser-places.js 847 PTH_init
chrome://navigator/content/navigator.js 664 Startup
chrome://navigator/content/navigator.xul 1 onload
null 0 null

Fichier Source : resource:///modules/Deprecated.jsm
Ligne : 79
}
When starting SeaMonkey. (Maybe 1 warning per browser window !!?)

Probably need to port:
Bug 969318 - "A promise chain failed to handle a rejection" from the livemarks service (deprecate passing a callback)

Comment 3

[Nerdine Zermok]

Philip, I'm not sure this issue is a duplicate,
and this problem persists from the last version of seamonkey
I use a target form to a window.open name, this issue does not appear on Firefox though

Comment 4

[Nerdine Zermok]

here is what I still get with seamonkey version 2.35
Timestamp: 9/25/2015 2:23:01 AM
Error: DEPRECATION WARNING: Passing a callback to Livermarks methods is deprecated. Please use the returned promise instead.
You may find more details about this deprecation at: https://developer.mozilla.org/docs/Mozilla/JavaScript_code_modules/Promise.jsm
resource://gre/components/nsLivemarkService.js 371 LS_getLivemark
chrome://communicator/content/places/browserPlacesViews.js 659 PVB_containerStateChanged
chrome://communicator/content/places/browserPlacesViews.js 75 PlacesViewBase.prototype.result
chrome://communicator/content/places/browserPlacesViews.js 49 PlacesViewBase.prototype.place
chrome://communicator/content/places/browserPlacesViews.js 13 PlacesViewBase
chrome://communicator/content/places/browserPlacesViews.js 857 PlacesToolbar
chrome://communicator/content/bookmarks/browser-places.js 847 PTH_init
chrome://navigator/content/navigator.js 664 Startup
chrome://navigator/content/navigator.xul 1 onload
https://secure.live-school.net/boss/index.lol 175 openTool
https://secure.live-school.net/boss/index.lol 1 onclick
null 0 null

Source File: resource://gre/modules/Deprecated.jsm
Line: 79


this is not reproduced on all other browsers (Firefox, Waterfox, Opera, IE, Chrome, Safari, Konqueror)

Comment 5

[Nerdine Zermok]

it seems that this error is related when a window.open() javascript is done with
arguments that customize the window like window.open('url','name','params=a,params=b');
if I remove all the params the error does not show.