It's akin to using nested tables for display, fer chrissakes!
Reasons why it's wrong:
- It's a poor user experience because the user doesn't get any of the usual browser functionality in the pop-up window, like the back button or the icon that tells you the page is loading.
- I'm sure I'll think of some more later