i'm dipping into embedding again. while trying to write a js proxy component, i've started to add some debugging statements in the c++ windowwatcher class (which i need to proxy). Are there any plans to resurrect the code after the return, or can I commit a patch to remove it?
Comment on attachment 74141 [details] [diff] [review] remove unreachable code email@example.com
Attachment #74141 - Flags: review+
Created attachment 74194 [details] [diff] [review] remove unused method better: get rid of the whole method and clean up a couple of misleading comments while you're at it.
Attachment #74141 - Attachment is obsolete: true
Comment on attachment 74194 [details] [diff] [review] remove unused method r=timeless
Attachment #74194 - Flags: review+
Comment on attachment 74194 [details] [diff] [review] remove unused method sr=jst if you make the GetWebBrowser() methods null out the out parameter.
Attachment #74194 - Flags: superreview+
Status: NEW → ASSIGNED
Summary: scary code → remove unused scary code that's bothering people
Target Milestone: --- → mozilla1.0
Comment on attachment 74194 [details] [diff] [review] remove unused method a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #74194 - Flags: approval+
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
nsWindowWatcher::InitializeDocshell() removed. Verified against 3/18/02 Mozilla 0.9.9 debug build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.