Bug 1401706: Remove redundant UpdateValueDisplay call. r?bholley draft
authorEmilio Cobos Álvarez <emilio@crisal.io>
Fri, 22 Sep 2017 14:48:02 +0200
changeset 669421 e25706cc7425eaac7b17a58ad01d242bdbd13c0e
parent 669420 5d7206f080b53b28d7cede49a24093a283874afb
child 732949 cc95a676eb9ebbdff69fa7a94a1fc1592fcfd0d8
push id81322
push userbmo:emilio@crisal.io
push dateSat, 23 Sep 2017 02:02:53 +0000
reviewersbholley
bugs1401706
milestone58.0a1
Bug 1401706: Remove redundant UpdateValueDisplay call. r?bholley This was just moved from the old code, but we always call that function in CreateAnonymousContent, so no need to. MozReview-Commit-ID: 1zqmQM2KdpW Signed-off-by: Emilio Cobos Álvarez <emilio@crisal.io>
layout/forms/nsTextControlFrame.cpp
--- a/layout/forms/nsTextControlFrame.cpp
+++ b/layout/forms/nsTextControlFrame.cpp
@@ -478,17 +478,17 @@ nsTextControlFrame::CreateRootNode()
       classValue.AppendLiteral(" inherit-overflow");
     }
     classValue.AppendLiteral(" inherit-scroll-behavior");
   }
   nsresult rv = mRootNode->SetAttr(kNameSpaceID_None, nsGkAtoms::_class,
                                    classValue, false);
   NS_ENSURE_SUCCESS(rv, rv);
 
-  return UpdateValueDisplay(false);
+  return NS_OK;
 }
 
 void
 nsTextControlFrame::CreatePlaceholderIfNeeded()
 {
   MOZ_ASSERT(!mPlaceholderDiv);
 
   // Do we need a placeholder node?