Bug 682041 followup - Add the missing return statement; r=need-to-learn-C++
authorEhsan Akhgari <ehsan@mozilla.com>
Thu, 08 Sep 2011 20:08:40 -0400
changeset 78105 01dcbca466f30431b08727f6d75e64798af15381
parent 78104 53cc48edabba1ce7f1c642459ea8d6cbc2bf9894
child 78106 78e81382db7f7c33f116ffff0bd521bca2513ece
push id78
push userclegnitto@mozilla.com
push dateFri, 16 Dec 2011 17:32:24 +0000
treeherdermozilla-release@79d24e644fdd [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersneed-to-learn-C
bugs682041
milestone9.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 682041 followup - Add the missing return statement; r=need-to-learn-C++
layout/forms/nsComboboxControlFrame.cpp
--- a/layout/forms/nsComboboxControlFrame.cpp
+++ b/layout/forms/nsComboboxControlFrame.cpp
@@ -193,16 +193,17 @@ NS_IMETHODIMP DestroyWidgetRunnable::Run
 {
   nsIView* view = nsnull;
   nsIWidget* currentWidget = GetWidget(&view);
   // Make sure that we are destroying the same widget as what was requested
   // when the event was fired.
   if (view && mWidget && mWidget == currentWidget) {
     view->DestroyWidget();
   }
+  return NS_OK;
 }
 
 nsIWidget* DestroyWidgetRunnable::GetWidget(nsIView** aOutView) const
 {
   nsIFrame* primaryFrame = mCombobox->GetPrimaryFrame();
   nsIComboboxControlFrame* comboboxFrame = do_QueryFrame(primaryFrame);
   if (comboboxFrame) {
     nsIFrame* dropdown = comboboxFrame->GetDropDown();