Bug 1220827 - Replace question comment in nsHTMLDocument::Open with an explanatory comment to explain why we create a new Window. r=smaug

2015-10-05 18:34:46 +01:00
parent 4e44f75050
commit 6eb9490d6f
1 changed files with 3 additions and 2 deletions
--- a/dom/html/nsHTMLDocument.cpp
+++ b/dom/html/nsHTMLDocument.cpp
@@ -1610,8 +1610,9 @@ nsHTMLDocument::Open(JSContext* cx,
    }
 #endif

-    // Should this pass true for aForceReuseInnerWindow?
-    rv = window->SetNewDocument(this, nullptr, false);
+    // Per spec, we pass false here so that a new Window is created.
+    rv = window->SetNewDocument(this, nullptr,
+                                /* aForceReuseInnerWindow */ false);
    if (rv.Failed()) {
      return nullptr;
    }