Bug 1443425: Removed unused variable shellGrip. r=emilio
authormmr <mmrxyz@gmail.com>
Thu, 08 Mar 2018 23:34:14 +0530
changeset 407347 457704c7f810c7506af50ab82bfd76bbabf4c95a
parent 407346 d1a36af94ccd8800d7538a9f9510c39dbcc090f5
child 407348 9967aedb77babcddcb06a518e030c83538127eb1
push id100641
push userecoal95@gmail.com
push dateFri, 09 Mar 2018 11:17:25 +0000
treeherdermozilla-inbound@457704c7f810 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersemilio
bugs1443425
milestone60.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1443425: Removed unused variable shellGrip. r=emilio Reviewer note: `shell` is already a strong reference, so no need for it. MozReview-Commit-ID: 8IbcFOZxoAz
dom/base/nsContentSink.cpp
--- a/dom/base/nsContentSink.cpp
+++ b/dom/base/nsContentSink.cpp
@@ -1260,17 +1260,16 @@ nsContentSink::StartLayout(bool aIgnoreP
   mDocument->SetMayStartLayout(true);
   nsCOMPtr<nsIPresShell> shell = mDocument->GetShell();
   // Make sure we don't call Initialize() for a shell that has
   // already called it. This can happen when the layout frame for
   // an iframe is constructed *between* the Embed() call for the
   // docshell in the iframe, and the content sink's call to OpenBody().
   // (Bug 153815)
   if (shell && !shell->DidInitialize()) {
-    nsCOMPtr<nsIPresShell> shellGrip = shell;
     nsresult rv = shell->Initialize();
     if (NS_FAILED(rv)) {
       return;
     }
   }
 
   // If the document we are loading has a reference or it is a
   // frameset document, disable the scroll bars on the views.