Closed
Bug 1326210
Opened 8 years ago
Closed 8 years ago
Noticed in Bug 1265881 Error: TypeError: temp is null Source File: chrome://global/content/bindings/preferences.xml Line: 1227
Categories
(Toolkit :: Preferences, defect)
Toolkit
Preferences
Tracking
()
RESOLVED
FIXED
mozilla53
| Tracking | Status | |
|---|---|---|
| firefox53 | --- | fixed |
People
(Reporter: philip.chee, Assigned: philip.chee)
Details
Attachments
(1 file)
|
1.18 KB,
patch
|
mossop
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•8 years ago
|
||
Attachment #8822435 -
Flags: review?(dtownsend)
|
||
Comment 2•8 years ago
|
||
Comment on attachment 8822435 [details] [diff] [review]
Patch v1.0 null check temp first
Review of attachment 8822435 [details] [diff] [review]:
-----------------------------------------------------------------
Looks like the only way we can get into this state is if aStartElement is null, might be nicer to just null check that up front. Not too bothered either way though.
Attachment #8822435 -
Flags: review?(dtownsend) → review+
Pushed by philip.chee@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4176cf985a19
Noticed in Bug 1265881 Error: TypeError: temp is null Source File: chrome://global/content/bindings/preferences.xml Line: 1227 r=Mossop
|
||
Comment 4•8 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in
before you can comment on or make changes to this bug.
Description
•