Expose UseCounter::Feature enum out of blink as WebFeature

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CSSParserContext_h
 #define CSSParserContext_h
 
 #include "core/CoreExport.h"
 #include "core/css/parser/CSSParserMode.h"
 #include "core/dom/Document.h"
@@ skipping 64 matching lines @@
     return use_legacy_background_size_shorthand_behavior_;
   }
 
   // FIXME: This setter shouldn't exist, however the current lifetime of
   // CSSParserContext is not well understood and thus we sometimes need to
   // override this field.
   void SetMode(CSSParserMode mode) { mode_ = mode; }
 
   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;
 
   ContentSecurityPolicyDisposition ShouldCheckContentSecurityPolicy() const {
     return should_check_content_security_policy_;
   }
 
   DECLARE_TRACE();
 
  private:
@@ skipping 19 matching lines @@
   ContentSecurityPolicyDisposition should_check_content_security_policy_;
 
   WeakMember<const Document> document_;
 };
 
 CORE_EXPORT const CSSParserContext* StrictCSSParserContext();
 
 }  // namespace blink
 
 #endif  // CSSParserContext_h