Worker: Enable UseCounter for SharedWorkerGlobalScope

content/browser/shared_worker/shared_worker_service_impl.cc

 // Copyright 2014 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.
 
 #include "content/browser/shared_worker/shared_worker_service_impl.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <iterator>
@@ skipping 329 matching lines @@
     SharedWorkerMessageFilter* filter,
     unsigned long long document_id) {
   ScopedWorkerDependencyChecker checker(this);
   for (WorkerHostMap::const_iterator iter = worker_hosts_.begin();
        iter != worker_hosts_.end();
        ++iter) {
     iter->second->DocumentDetached(filter, document_id);
   }
 }
 
+void SharedWorkerServiceImpl::CountFeature(SharedWorkerMessageFilter* filter,
+                                           int worker_route_id,
+                                           uint32_t feature) {
+  ScopedWorkerDependencyChecker checker(this);

[horo, 2017/01/25 03:45:10]

You don't need this?

[nhiroki, 2017/01/25 10:16:11]

Done.

+  if (SharedWorkerHost* host =
+          FindSharedWorkerHost(filter->render_process_id(), worker_route_id))

[horo, 2017/01/25 03:45:10]

nit: brackets

[nhiroki, 2017/01/25 10:16:11]

Done.

+    host->CountFeature(feature);
+}
+
 void SharedWorkerServiceImpl::WorkerContextClosed(
     SharedWorkerMessageFilter* filter,
     int worker_route_id) {
   ScopedWorkerDependencyChecker checker(this);
   if (SharedWorkerHost* host =
           FindSharedWorkerHost(filter->render_process_id(), worker_route_id))
     host->WorkerContextClosed();
 }
 
 void SharedWorkerServiceImpl::WorkerContextDestroyed(
     SharedWorkerMessageFilter* filter,
     int worker_route_id) {
   ScopedWorkerDependencyChecker checker(this);
+  if (SharedWorkerHost* host =
+          FindSharedWorkerHost(filter->render_process_id(), worker_route_id))

[horo, 2017/01/25 03:45:10]

nit: brackets

[nhiroki, 2017/01/25 10:16:11]

Done.

+    host->WorkerContextDestroyed();
   ProcessRouteIdPair key(filter->render_process_id(), worker_route_id);
   worker_hosts_.erase(key);
 }
 
 void SharedWorkerServiceImpl::WorkerReadyForInspection(
     SharedWorkerMessageFilter* filter,
     int worker_route_id) {
   if (SharedWorkerHost* host =
           FindSharedWorkerHost(filter->render_process_id(), worker_route_id))
     host->WorkerReadyForInspection();
@@ skipping 293 matching lines @@
     UpdateWorkerDependencyFunc new_func) {
   update_worker_dependency_ = new_func;
 }
 
 void SharedWorkerServiceImpl::ChangeTryIncrementWorkerRefCountFuncForTesting(
     bool (*new_func)(int)) {
   s_try_increment_worker_ref_count_ = new_func;
 }
 
 }  // namespace content