Worker: Connect UseCounter to workers and worklets

third_party/WebKit/Source/core/workers/ThreadedWorkletGlobalScope.cpp

Index: third_party/WebKit/Source/core/workers/ThreadedWorkletGlobalScope.cpp
diff --git a/third_party/WebKit/Source/core/workers/ThreadedWorkletGlobalScope.cpp b/third_party/WebKit/Source/core/workers/ThreadedWorkletGlobalScope.cpp
index 0be4de585a9228d46101cadd2d80e1926122fe35..3a770c4a007d7e58febbfa9121b3067206b12075 100644
--- a/third_party/WebKit/Source/core/workers/ThreadedWorkletGlobalScope.cpp
+++ b/third_party/WebKit/Source/core/workers/ThreadedWorkletGlobalScope.cpp
@@ -30,6 +30,22 @@ ThreadedWorkletGlobalScope::~ThreadedWorkletGlobalScope() {
   DCHECK(!m_thread);
 }
 
+void ThreadedWorkletGlobalScope::countFeature(
+    UseCounter::Feature feature) const {
+  DCHECK(isContextThread());
+  DCHECK(m_thread);
+  WorkletGlobalScope::countFeature(feature);
+  m_thread->workerReportingProxy().countFeature(feature);

[falken, 2016/12/02 08:54:49]

Why are both calls needed? This doesn't double count?

[nhiroki, 2016/12/05 04:49:01]

This doesn't double count because (WorkerOr)WorkletGlobalScope::countFeature()
does nothing.

I meant to call WorkletGlobalScope::countFeature() for putting common operations
in ancestors' countFeature() like countDeprecation(), but it's not necessary for
now. I made it pure-virtual function and removed this call.

+}
+
+void ThreadedWorkletGlobalScope::countDeprecation(
+    UseCounter::Feature feature) const {
+  DCHECK(isContextThread());
+  DCHECK(m_thread);
+  WorkletGlobalScope::countDeprecation(feature);
+  m_thread->workerReportingProxy().countDeprecation(feature);
+}
+
 void ThreadedWorkletGlobalScope::dispose() {
   DCHECK(isContextThread());
   WorkletGlobalScope::dispose();