Bug 1408312 - Part 2: Replace nsCSSParser usage in inDOMUtils::IsValidCSSColor. r?xidorn draft
authorCameron McCormack <cam@mcc.id.au>
Mon, 16 Oct 2017 09:02:35 +0800
changeset 680688 d90d09ecc641c3fb7727c0d834ff392d8d1f657f
parent 680687 c4b36b684948508943322cd427a4a7cf7d85ab16
child 680689 ce337b933fac479405a76d8c9bca85d262619769
push id84585
push userbmo:cam@mcc.id.au
push dateMon, 16 Oct 2017 03:56:44 +0000
reviewersxidorn
bugs1408312
milestone58.0a1
Bug 1408312 - Part 2: Replace nsCSSParser usage in inDOMUtils::IsValidCSSColor. r?xidorn MozReview-Commit-ID: AGo249i7ATL
layout/inspector/inDOMUtils.cpp
--- a/layout/inspector/inDOMUtils.cpp
+++ b/layout/inspector/inDOMUtils.cpp
@@ -47,16 +47,17 @@
 #include "nsCSSValue.h"
 #include "nsColor.h"
 #include "mozilla/StyleSetHandleInlines.h"
 #include "nsStyleUtil.h"
 #include "nsQueryObject.h"
 #include "mozilla/ServoBindings.h"
 #include "mozilla/ServoStyleRule.h"
 #include "mozilla/ServoStyleRuleMap.h"
+#include "mozilla/ServoCSSParser.h"
 
 using namespace mozilla;
 using namespace mozilla::css;
 using namespace mozilla::dom;
 
 ///////////////////////////////////////////////////////////////////////////////
 
 inDOMUtils::inDOMUtils()
@@ -1031,19 +1032,17 @@ inDOMUtils::ColorToRGBA(const nsAString&
   }
 
   return NS_OK;
 }
 
 NS_IMETHODIMP
 inDOMUtils::IsValidCSSColor(const nsAString& aColorString, bool *_retval)
 {
-  nsCSSParser cssParser;
-  nsCSSValue cssValue;
-  *_retval = cssParser.ParseColorString(aColorString, nullptr, 0, cssValue, true);
+  *_retval = ServoCSSParser::IsValidCSSColor(aColorString);
   return NS_OK;
 }
 
 NS_IMETHODIMP
 inDOMUtils::CssPropertyIsValid(const nsAString& aPropertyName,
                                const nsAString& aPropertyValue,
                                bool *_retval)
 {