Closed Bug 1326210 Opened 7 years ago Closed 7 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)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: philip.chee, Assigned: philip.chee)

Details

Attachments

(1 file)

      No description provided.
Attachment #8822435 - Flags: review?(dtownsend)
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
https://hg.mozilla.org/mozilla-central/rev/4176cf985a19
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: