1706004 - Make not refer to document.popupNode in Places.

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, task, P5)

Description

[Daisuke Akatsuka (:daisuke)]

Currently, we are referring to document.popupNode in Places to make context menus in some places. But, as the field is already deprecated, we should not use it.
In this bug, we will replace the places using document.popupNode with menupopup.triggerNode.

Comment 1

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

The code has been removed, there is only one comment about document.popupNode at https://searchfox.org/mozilla-central/source/browser/components/places/PlacesUIUtils.jsm#464-465

Comment 2

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

Thisi s a trivial comment change bug to check the build environment is working.

Comment 3

[Leslie]

Hi there, I am an outreachy applicant and would like to work on this bug. Can I be assigned to it?

Comment 4

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

Feel free to start working on it, bugs are assigned when the first patch is submitted. Thank you.

Comment 5

[Leslie]

Attachment: Bug 1706004 - Delete reference to document.popupNode in Places. r?mak

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/33152631ec40
Delete reference to document.popupNode in Places. r=mak

Comment 7

[Marco Bonardo [:mak] (Away Apr 25 - May 5)]

Please remember to record your contribution on Outreachy.

Comment 8

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/33152631ec40