Warning: redeclaration of var loadInBackground Source File: chrome://communicator/content/contentAreaClick.js Line: 227, Column: 14 Source Code: var loadInBackground = pref.getBoolPref("browser.tabs.loadInBackground"); This was caused by caillon's patch in bug 113072. Patch for that coming up in a few minutes.
Reassigning to caillon as requested. I have a patch for the strict warning coming up in a few seconds.
Assignee: jaggernaut → caillon
Comment on attachment 66529 [details] [diff] [review] patch for strict warning no, I need to remove that variable as jag and I have talked about. The nifty patch I had on bug 113072 is closer to what this should be. aside from that, if I weren't going to remove this var, this is still the wrong way to fix this. The variable should be declared before the switch, really, not like this. I'll add a patch to this shortly.
Attachment #66529 - Flags: needs-work+
17 years ago
Status: NEW → ASSIGNED
OS: Windows 98 → All
Priority: -- → P4
Hardware: PC → All
Target Milestone: --- → mozilla0.9.9
Created attachment 70465 [details] [diff] [review] Remove strict warn
Attachment #66529 - Attachment is obsolete: true
Comment on attachment 70465 [details] [diff] [review] Remove strict warn sr=jag
Attachment #70465 - Flags: superreview+
Comment on attachment 70465 [details] [diff] [review] Remove strict warn r=walk84
Attachment #70465 - Flags: review+
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
no longer see this js strict warning... tested with 2002.07.23-1.0 bits.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.