Closed
Bug 1555908
Opened 6 years ago
Closed 6 years ago
Remove event.target null checks in tabbrowser.xml
Categories
(Firefox :: Tabbed Browser, defect, P1)
Firefox
Tabbed Browser
Tracking
()
RESOLVED
FIXED
Firefox 69
Tracking | Status | |
---|---|---|
firefox69 | --- | fixed |
People
(Reporter: dao, Assigned: surkov)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
Bug 1519514 added some null checks for event.target in tabbrowser.xml. However, event.target cannot be null.
Alexander, could you please look into cleaning this up?
Flags: needinfo?(surkov.alexander)
Assignee | ||
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
Type: task → defect
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → surkov.alexander
Type: defect → task
Flags: needinfo?(surkov.alexander)
Reporter | ||
Comment 3•6 years ago
|
||
(In reply to Release mgmt bot [:sylvestre / :calixte] from comment #2)
Bugbug thinks this bug is a defect, but please change it back in case of error.
This was basically wrong, but as it turns out now there's potentially a bug here that we should fix as part of this: https://phabricator.services.mozilla.com/D33266#inline-197651
Type: task → defect
Priority: P3 → P1
Pushed by asurkov@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f5e4b365f02e
Remove event.target null checks in tabbrowser.xml, r=dao
![]() |
||
Comment 5•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 69
Updated•3 years ago
|
Has Regression Range: --- → yes
Updated•3 years ago
|
Keywords: regression
You need to log in
before you can comment on or make changes to this bug.
Description
•