Expose UseCounter::Feature enum out of blink as WebFeature

third_party/WebKit/Source/core/css/parser/CSSParserContext.h

Index: third_party/WebKit/Source/core/css/parser/CSSParserContext.h
diff --git a/third_party/WebKit/Source/core/css/parser/CSSParserContext.h b/third_party/WebKit/Source/core/css/parser/CSSParserContext.h
index 2e3b2ae618d827c8d0d878141ab7b2fecda18024..fe628d1f0485625517d0cc29f0c2f6c277916f9f 100644
--- a/third_party/WebKit/Source/core/css/parser/CSSParserContext.h
+++ b/third_party/WebKit/Source/core/css/parser/CSSParserContext.h
@@ -82,9 +82,17 @@ class CORE_EXPORT CSSParserContext
 
   KURL CompleteURL(const String& url) const;
 
+  // TODO(lunalu): Deprecate UseCounter::Feature by WebFeature in Count() and
+  // CounterDeprecation().
   void Count(UseCounter::Feature) const;
+  void Count(WebFeature feature) const {
+    return Count(static_cast<UseCounter::Feature>(feature));

[Rick Byers, 2017/05/23 19:51:49]

I think this would be slightly cleaner if you inverted the relationship here. 
I.e. update the implementation of 'Count' to use WebFeature, and have the
UseCounter::Feature one delegate to the WebFeature overload.

This is probably also technically more correct - as some new values may get
added to WebFeature before UseCounter::Feature is removed, so WebFeature will be
a strict superset of the values in UseCounter::Feature.

[lunalu1, 2017/05/24 20:56:32]

Right. Reverted.

+  }
   void Count(CSSParserMode, CSSPropertyID) const;
   void CountDeprecation(UseCounter::Feature) const;
+  void CountDeprecation(WebFeature feature) const {
+    return CountDeprecation(static_cast<UseCounter::Feature>(feature));
+  }
   bool IsUseCounterRecordingEnabled() const { return document_; }
   bool IsDocumentHandleEqual(const Document* other) const;