Bug 1483127 - Use NS_IMETHODIMP at definition of HTMLEditor::SetIsCSSEnabled() r=m_kato
authorMasayuki Nakano <masayuki@d-toybox.com>
Thu, 16 Aug 2018 10:05:06 +0000
changeset 432043 ef882e3e4b6bb67e0bfef77939d1bd1addc68998
parent 432042 c7cba947e5046220f9a3678b465df6db769c58f7
child 432044 fd90e385d65bc4eeb960adaae509458504bbead8
push id34457
push userebalazs@mozilla.com
push dateFri, 17 Aug 2018 09:45:26 +0000
treeherdermozilla-central@d2ba1d6c76f2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersm_kato
bugs1483127
milestone63.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 1483127 - Use NS_IMETHODIMP at definition of HTMLEditor::SetIsCSSEnabled() r=m_kato HTMLEditor::SetIsCSSEnabled() is an XPCOM but it's defined with nsresult. Differential Revision: https://phabricator.services.mozilla.com/D3459
editor/libeditor/HTMLEditor.cpp
--- a/editor/libeditor/HTMLEditor.cpp
+++ b/editor/libeditor/HTMLEditor.cpp
@@ -4395,17 +4395,17 @@ HTMLEditor::RemoveAttributeOrEquivalent(
     return NS_OK;
   }
 
   return aSuppressTransaction ?
     aElement->UnsetAttr(kNameSpaceID_None, aAttribute, /* aNotify = */ true) :
     RemoveAttributeWithTransaction(*aElement, *aAttribute);
 }
 
-nsresult
+NS_IMETHODIMP
 HTMLEditor::SetIsCSSEnabled(bool aIsCSSPrefChecked)
 {
   if (!mCSSEditUtils) {
     return NS_ERROR_NOT_INITIALIZED;
   }
 
   mCSSEditUtils->SetCSSEnabled(aIsCSSPrefChecked);