Closed Bug 228619 Opened 21 years ago Closed 21 years ago

Warning ``assignment to undeclared variable entry'' [xs] in file ``chrome://communicator/content/contentAreaUtils.js'', line 667, character 0.

Categories

(SeaMonkey :: UI Design, defect)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: timeless, Assigned: timeless)

References

Details

Attachments

(2 obsolete files)

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6b) Gecko/20031208 Warning ``assignment to undeclared variable entry'' [xs] in file ``chrome://communicator/content/contentAreaUtils.js'', line 667, character 0. Stopped for error handler. #0: function getPostData() in <chrome://communicator/content/contentAreaUtils.js> line 667 665: try { 666: var sessionHistory = getWebNavigation().sessionHistory; 667: entry = sessionHistory.getEntryAtIndex(sessionHistory.index, false); 668: entry = entry.QueryInterface(Components.interfaces.nsISHEntry); 669: return entry.postData;
Attachment #137501 - Flags: superreview?(bz-vacation)
Attachment #137501 - Flags: review?(neil.parkwaycc.co.uk)
Comment on attachment 137501 [details] [diff] [review] (Av1) skip |entry| [Checked in: Comment 4] Sure. sr=bzbarsky
Attachment #137501 - Flags: superreview?(bz-vacation) → superreview+
Comment on attachment 137501 [details] [diff] [review] (Av1) skip |entry| [Checked in: Comment 4] I'm surprised you didn't inline sessionHistory too ;-)
Attachment #137501 - Flags: review?(neil.parkwaycc.co.uk) → review+
Comment on attachment 137501 [details] [diff] [review] (Av1) skip |entry| [Checked in: Comment 4] Check in: { 12/20/2003 20:06 timeless%mozdev.org 1.101 }
Attachment #137501 - Attachment description: skip the variable → skip the variable [Checked in: Comment 3]
Attachment #137501 - Attachment is obsolete: true
Attachment #137501 - Attachment description: skip the variable [Checked in: Comment 3] → (Av1) skip the variable [Checked in: Comment 3]
Attachment #137501 - Attachment description: (Av1) skip the variable [Checked in: Comment 3] → (Av1) skip the variable [Checked in: Comment 4]
Attachment #137501 - Attachment description: (Av1) skip the variable [Checked in: Comment 4] → (Av1) skip |entry| [Checked in: Comment 4]
Attachment #137784 - Flags: superreview?(bz-vacation)
Attachment #137784 - Flags: review?(neil.parkwaycc.co.uk)
OS: Windows XP → All
Hardware: PC → All
Attachment #137784 - Flags: review?(neil.parkwaycc.co.uk) → review+
Comment on attachment 137784 [details] [diff] [review] (Bv1) skip |sessionHistory| too [Checked in: Comment 8] sr=bzbarsky
Attachment #137784 - Flags: superreview?(bz-vacation) → superreview+
Fix checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment on attachment 137784 [details] [diff] [review] (Bv1) skip |sessionHistory| too [Checked in: Comment 8] Check in: { 12/25/2003 03:50 neil%parkwaycc.co.uk 1.102 }
Attachment #137784 - Attachment description: (Bv1) skip |sessionHistory| too → (Bv1) skip |sessionHistory| too [Checked in: Comment 8]
Attachment #137784 - Attachment is obsolete: true
Comment on attachment 137784 [details] [diff] [review] (Bv1) skip |sessionHistory| too [Checked in: Comment 8] This caused bug 242261.
Product: Core → Mozilla Application Suite
Component: XP Apps: GUI Features → UI Design
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: