Closed Bug 966498 Opened 6 years ago Closed 6 years ago
Clean up ns
Global Window::Alert a little bit
nsGlobalWindow::Alert and nsGlobalWindow::Confirm are almost exactly the same and both contain a really ugly (and unnecessary!) reinterpret_cast<void**>. It's pretty easy to merge them and fix the reinterpret_cast. I'm purposely leaving nsGlobalWindow::Prompt out of the refactoring because it's just different enough.
This was pretty straightforward...
Attachment #8368876 - Flags: review?(bzbarsky)
The only thing to note here is that outerwindow->mDoc == innerwindow->mDoc
Attachment #8368878 - Flags: review?(bzbarsky)
Huh. Git's patch formatter totally did better locally...
Assignee: nobody → mrbkap
Comment on attachment 8368876 [details] [diff] [review] Refactor r=me
Attachment #8368876 - Flags: review?(bzbarsky) → review+
Comment on attachment 8368878 [details] [diff] [review] A little more cleanup r=me
Attachment #8368878 - Flags: review?(bzbarsky) → review+
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.