Add use counter for chrome.loadTimes

chrome/renderer/loadtimes_extension_bindings.cc

 // Copyright (c) 2012 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 "chrome/renderer/loadtimes_extension_bindings.h"
 
 #include <math.h>
 
 #include "base/time/time.h"
 #include "content/public/renderer/document_state.h"
@@ skipping 91 matching lines @@
         return kTransitionReload;
       case blink::WebNavigationTypeOther:
         return kTransitionOther;
     }
     return kTransitionOther;
   }
 
   static void LoadtimesGetter(
       v8::Local<v8::Name> name,
       const v8::PropertyCallbackInfo<v8::Value>& info) {
-    // TODO(panicker): Add Usage counter.
+    WebLocalFrame* frame = WebLocalFrame::frameForCurrentContext();
+    if (frame) {

[esprehn, 2016/08/05 20:57:36]

I'd usually write:

if (WebLocalFrame* frame = WebLocalFrame::frameForCurrentContext()) {
  // ...
}

but you can do this if you want :)

[panicker, 2016/08/05 23:06:14]

Sure, updated.

+      frame->usageCountChromeLoadtimes(blink::WebString::fromUTF8(
+          *v8::String::Utf8Value(name)));
+    }
     info.GetReturnValue().Set(info.Data());
   }
 
   static void GetLoadTimes(const v8::FunctionCallbackInfo<v8::Value>& args) {
     args.GetReturnValue().SetNull();
     WebLocalFrame* frame = WebLocalFrame::frameForCurrentContext();
     if (!frame) {
       return;
     }
     WebDataSource* data_source = frame->dataSource();
@@ skipping 268 matching lines @@
     }
     args.GetReturnValue().Set(csi);
   }
 };
 
 v8::Extension* LoadTimesExtension::Get() {
   return new LoadTimesExtensionWrapper();
 }
 
 }  // namespace extensions_v8