Closed Bug 1307235 Opened 8 years ago Closed 8 years ago

"ReferenceError: Utils is not defined" in SessionStore.jsm

Categories

(Firefox for Android Graveyard :: General, defect)

All
Android
defect
Not set
normal

Tracking

(firefox52 fixed)

RESOLVED FIXED
Firefox 52
Tracking Status
firefox52 --- fixed

People

(Reporter: jchen, Assigned: ckerschb)

References

Details

Attachments

(1 file)

There are a lot of "ReferenceError: Utils is not defined" messages in adb logcat, coming from SessionStore.jsm (e.g. [1]). I think bug 1297338 forgot to import "resource:///modules/sessionstore/Utils.jsm".

[1] https://dxr.mozilla.org/mozilla-central/rev/955840bfd3c20eb24dd5a01be27bdc55c489a285/mobile/android/components/SessionStore.js#1215
Flags: needinfo?(ckerschb)
You are right, thanks for filing the bug!
Assignee: nobody → ckerschb
Status: NEW → ASSIGNED
Flags: needinfo?(ckerschb)
Attachment #8797411 - Flags: review?(nchen)
Attachment #8797411 - Flags: review?(nchen) → review+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/cf23f8fe42fb
Import sessionstore/Utils into android/components/SessionStore. r=jchen
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/cf23f8fe42fb
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Did this patch help? I still see lots of "Utils is not defined" in test logcats. How can we find where this is coming from?

http://mozilla-releng-blobs.s3.amazonaws.com/blobs/mozilla-inbound/sha512/5c60170691c950f2daf3f9b2e8ecddcba5532f8d756d5eaabaedb7293d6903bbd3a3b0fca17f24a3710cfe0581d697fd0b8977a6c65111175e511ef9236607db
Flags: needinfo?(nchen)
Flags: needinfo?(ckerschb)
Let's sort this out in bug 1313397.
Flags: needinfo?(nchen)
Flags: needinfo?(ckerschb)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: