Expose UseCounter::Feature enum out of blink as WebFeature

third_party/WebKit/Source/core/loader/BaseFetchContext.h

 // Copyright 2017 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 BaseFetchContext_h
 #define BaseFetchContext_h
 
 #include "core/CoreExport.h"
 #include "core/frame/UseCounter.h"
 #include "core/frame/csp/ContentSecurityPolicy.h"
@@ skipping 54 matching lines @@
   // and AllowResponse.
   virtual bool ShouldBlockRequestByInspector(const ResourceRequest&) const = 0;
   virtual void DispatchDidBlockRequest(const ResourceRequest&,
                                        const FetchInitiatorInfo&,
                                        ResourceRequestBlockedReason) const = 0;
   // TODO(kinuko): Consider implementing this on ExecutionContext and
   // remove this virtual method.
   virtual void ReportLocalLoadFailed(const KURL&) const = 0;
   virtual bool ShouldBypassMainWorldCSP() const = 0;
   virtual bool IsSVGImageChromeClient() const = 0;
+  // TODO(lunalu): Deprecate UseCounter::Feature by WebFeature in

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

From a quick scan I think all the worker-related support could also be switched
over to WebFeature without having to update a ton of callsites.  The duplicate
call-site thing is really only helpful in the cases where there are hundreds of
callers.

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

Reverted.

+  // CountDeprecation() and CountUsage().
   virtual void CountUsage(UseCounter::Feature) const = 0;
+  virtual void CountUsage(WebFeature) const = 0;
   virtual void CountDeprecation(UseCounter::Feature) const = 0;
+  virtual void CountDeprecation(WebFeature) const = 0;
   virtual bool ShouldBlockFetchByMixedContentCheck(
       const ResourceRequest&,
       const KURL&,
       SecurityViolationReportingPolicy) const = 0;
 
   // Utility method that can be used to implement other methods.
   void PrintAccessDeniedMessage(const KURL&) const;
   void AddCSPHeaderIfNecessary(Resource::Type, ResourceRequest&);
   ResourceRequestBlockedReason CheckCSPForRequest(
       const ResourceRequest&,
@@ skipping 16 matching lines @@
 
   // FIXME: Oilpan: Ideally this should just be a traced Member but that will
   // currently leak because ComputedStyle and its data are not on the heap.
   // See crbug.com/383860 for details.
   WeakMember<ExecutionContext> execution_context_;
 };
 
 }  // namespace blink
 
 #endif  // BaseFetchContext_h