context menus break if someone decides they don't want sidebarOverlay.xul since we include the contentAreaContextOverlay from there. We should remove the hidden dependency. Patch forthcoming...
Created attachment 89364 [details] [diff] [review] Patch Simply adds the contentAreaContextOverlay include into the files that were including the sidebarOverlay.xul already and removes the dependency from the sidebarOverlay.
Comment on attachment 89364 [details] [diff] [review] Patch r=sgehani Could you list the tests conducted?
Attachment #89364 - Flags: review+
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla1.1beta
Blake, could you sr= this?
Comment on attachment 89364 [details] [diff] [review] Patch sr=jag
Attachment #89364 - Flags: superreview+
Landed on the trunk
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.