Bug 1261842 - ContentParent::RecvCreateWindow should always set an nsresult outparam. r=smaug

MozReview-Commit-ID: 79FYAMBIDef

--HG--
extra : rebase_source : d773574c50780bbe14639fb197f9148855c25892
extra : source : e2472a4cc209e0be8a083dd254965598d41f53c6

dom/ipc/ContentParent.cpp

@@ -5295,6 +5295,7 @@ ContentParent::RecvCreateWindow(PBrowserParent* aThisTab,
 {
   // We always expect to open a new window here. If we don't, it's an error.
   *aWindowIsNew = true;
+  *aResult = NS_OK;
 
   // The content process should never be in charge of computing whether or
   // not a window should be private or remote - the parent will do that.