forked from mirrors/gecko-dev
		
	Bug 1653103 - Part 1: Make nsWebBrowser::InternalDestroy return void; r=nika
Differential Revision: https://phabricator.services.mozilla.com/D83706
This commit is contained in:
		
							parent
							
								
									53eace3ac3
								
							
						
					
					
						commit
						2ef1036d17
					
				
					 2 changed files with 2 additions and 5 deletions
				
			
		| 
						 | 
					@ -170,8 +170,7 @@ already_AddRefed<nsWebBrowser> nsWebBrowser::Create(
 | 
				
			||||||
  return browser.forget();
 | 
					  return browser.forget();
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
NS_IMETHODIMP
 | 
					void nsWebBrowser::InternalDestroy() {
 | 
				
			||||||
nsWebBrowser::InternalDestroy() {
 | 
					 | 
				
			||||||
  if (mInternalWidget) {
 | 
					  if (mInternalWidget) {
 | 
				
			||||||
    mInternalWidget->SetWidgetListener(nullptr);
 | 
					    mInternalWidget->SetWidgetListener(nullptr);
 | 
				
			||||||
    mInternalWidget->Destroy();
 | 
					    mInternalWidget->Destroy();
 | 
				
			||||||
| 
						 | 
					@ -184,8 +183,6 @@ nsWebBrowser::InternalDestroy() {
 | 
				
			||||||
    mDocShellTreeOwner->WebBrowser(nullptr);
 | 
					    mDocShellTreeOwner->WebBrowser(nullptr);
 | 
				
			||||||
    mDocShellTreeOwner = nullptr;
 | 
					    mDocShellTreeOwner = nullptr;
 | 
				
			||||||
  }
 | 
					  }
 | 
				
			||||||
 | 
					 | 
				
			||||||
  return NS_OK;
 | 
					 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
NS_IMPL_CYCLE_COLLECTING_ADDREF(nsWebBrowser)
 | 
					NS_IMPL_CYCLE_COLLECTING_ADDREF(nsWebBrowser)
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -113,7 +113,7 @@ class nsWebBrowser final : public nsIWebBrowser,
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 protected:
 | 
					 protected:
 | 
				
			||||||
  virtual ~nsWebBrowser();
 | 
					  virtual ~nsWebBrowser();
 | 
				
			||||||
  NS_IMETHOD InternalDestroy();
 | 
					  void InternalDestroy();
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  // XXXbz why are these NS_IMETHOD?  They're not interface methods!
 | 
					  // XXXbz why are these NS_IMETHOD?  They're not interface methods!
 | 
				
			||||||
  NS_IMETHOD SetDocShell(nsIDocShell* aDocShell);
 | 
					  NS_IMETHOD SetDocShell(nsIDocShell* aDocShell);
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue