Closed Bug 636637 Opened 14 years ago Closed 14 years ago

"e is not defined" error while trying to log "Error while loading thumbnail"

Categories

(Firefox Graveyard :: Panorama, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Unfocused, Assigned: ttaubert)

References

Details

Attachments

(1 file, 1 obsolete file)

Error: e is not defined Source file: chrome://browser/content/tabview.js Line: 1094 Which is this: https://mxr.mozilla.org/mozilla-central/source/browser/base/content/tabview/storage.js#295 This on a nightly build from a couple of days ago: Mozilla/5.0 (X11; Linux x86_64; rv:2.0b12pre) Gecko/20110222 Firefox/4.0b12pre (Which, I think, is the same as beta12)
Blocks: 585689
Assignee: nobody → tim.taubert
Status: NEW → ASSIGNED
OS: Linux → All
Hardware: x86_64 → All
Attached patch patch v1 (obsolete) — Splinter Review
Attachment #515045 - Flags: review?(ian)
Attachment #515045 - Flags: feedback?(mitcho)
Priority: -- → P3
Comment on attachment 515045 [details] [diff] [review] patch v1 Thanks Blair + Tim for catching this.
Attachment #515045 - Flags: feedback?(mitcho) → feedback+
Attachment #515045 - Flags: review?(ian) → review+
Comment on attachment 515045 [details] [diff] [review] patch v1 Note to approvers: Non-risk one-line change that eliminates the use of an undefined variable.
Attachment #515045 - Flags: approval2.0?
Comment on attachment 515045 [details] [diff] [review] patch v1 a=beltzner
Attachment #515045 - Flags: approval2.0? → approval2.0+
Attachment #515045 - Attachment is obsolete: true
Keywords: checkin-needed
Attachment #515733 - Attachment description: patch for checkin → [checked-in] patch for checkin
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: