Combine two if statements in UpdateTopLevelContentWindowIDHelper.jsm

RESOLVED FIXED in Firefox 57

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: kershaw, Assigned: ami.bugzilla)

Tracking

({good-first-bug})

unspecified
Firefox 57
good-first-bug
Points:
---

Firefox Tracking Flags

(firefox57 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Updated

2 years ago
Keywords: good-first-bug
(Assignee)

Comment 1

2 years ago
Hi, I'd like to take this bug, and here is my patch.
Thanks for reviewing this.
Attachment #8895240 - Flags: review?(dao+bmo)
Comment on attachment 8895240 [details] [diff] [review]
Combine two if statements

Thanks!
Attachment #8895240 - Flags: review?(dao+bmo) → review+
Assignee: nobody → ami.bugzilla
Keywords: checkin-needed

Comment 3

2 years ago
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/495a724af85c
Combine two if statements; r=dao
Keywords: checkin-needed

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/495a724af85c
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
You need to log in before you can comment on or make changes to this bug.