Replace mozMatchesSelector usage with matches in browser-places.js

RESOLVED FIXED in Firefox 63

Status

()

P3
normal
RESOLVED FIXED
4 months ago
4 months ago

People

(Reporter: miketaylr, Assigned: e7358d9c)

Tracking

(Blocks: 1 bug)

57 Branch
Firefox 63
Points:
---

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 months ago
https://searchfox.org/mozilla-central/source/browser/base/content/browser-places.js#175

> if (!this.panel.mozMatchesSelector(":hover")) { 

should be changed to:

> if (!this.panel.matches(":hover")) {
Priority: -- → P3
(Assignee)

Comment 1

4 months ago
Created attachment 9002403 [details]
Bug 1482498 - Remove mozMatchesSelector(…) usage from browser‑places.js
(Assignee)

Updated

4 months ago
Assignee: nobody → e7358d9c
(Reporter)

Comment 2

4 months ago
Comment on attachment 9002403 [details]
Bug 1482498 - Remove mozMatchesSelector(…) usage from browser‑places.js

Mike Taylor [:miketaylr] (62 Regression Engineering Owner)(PTO Aug 15 - 17) has approved the revision.
Attachment #9002403 - Flags: review+
(Assignee)

Updated

4 months ago
Keywords: checkin-needed

Comment 3

4 months ago
Pushed by dvarga@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/32a3585e5b0b
Remove mozMatchesSelector(…) usage from browser‑places.js r=miketaylr
Keywords: checkin-needed

Comment 4

4 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/32a3585e5b0b
Status: NEW → RESOLVED
Last Resolved: 4 months ago
status-firefox63: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
You need to log in before you can comment on or make changes to this bug.