Closed Bug 1260116 Opened 4 years ago Closed 4 years ago

Replace String.prototype.contains with String.prototype.includes in debugger.js

Categories

(DevTools :: Debugger, defect, P3)

defect

Tracking

(firefox48 fixed)

RESOLVED FIXED
Firefox 48
Tracking Status
firefox48 --- fixed

People

(Reporter: jlast, Assigned: jlast)

Details

Attachments

(1 file)

String.prototype.contains was renamed to includes recently. Bug 1102219.
Attached patch includes.patchSplinter Review
Attachment #8735439 - Flags: review?(jlong)
Comment on attachment 8735439 [details] [diff] [review]
includes.patch

Review of attachment 8735439 [details] [diff] [review]:
-----------------------------------------------------------------

Nice!

Don't forget to assign yourself to this bug.
Attachment #8735439 - Flags: review?(jlong) → review+
Moar cleanup! Assigning P3 to this.
Priority: -- → P3
Assignee: nobody → jlaster
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/078fb73f17c2
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.