Pre-work for adding usage 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"
+#include "extensions/renderer/v8_helpers.h"
 #include "net/http/http_response_info.h"
 #include "third_party/WebKit/public/web/WebLocalFrame.h"
 #include "third_party/WebKit/public/web/WebPerformance.h"
 #include "v8/include/v8.h"
 
 using blink::WebDataSource;
 using blink::WebLocalFrame;
 using blink::WebNavigationType;
 using blink::WebPerformance;
 using content::DocumentState;
@@ skipping 77 matching lines @@
       case blink::WebNavigationTypeBackForward:
         return kTransitionForwardBack;
       case blink::WebNavigationTypeReload:
         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.
+    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();
     if (!data_source) {
       return;
     }
@@ skipping 39 matching lines @@
     std::string navigation_type =
         GetNavigationType(data_source->navigationType());
     bool was_fetched_via_spdy = document_state->was_fetched_via_spdy();
     bool was_npn_negotiated = document_state->was_npn_negotiated();
     std::string npn_negotiated_protocol =
         document_state->npn_negotiated_protocol();
     bool was_alternate_protocol_available =
         document_state->was_alternate_protocol_available();
     std::string connection_info = net::HttpResponseInfo::ConnectionInfoToString(
         document_state->connection_info());
+
     // Important: |frame|, |data_source| and |document_state| should not be
     // referred to below this line, as JS setters below can invalidate these
     // pointers.
     v8::Isolate* isolate = args.GetIsolate();
     v8::Local<v8::Context> ctx = isolate->GetCurrentContext();
-    v8::Local<v8::Object> load_times = v8::Object::New(isolate);
-    if (!load_times
-             ->Set(ctx, v8::String::NewFromUtf8(isolate, "requestTime",
-                                                v8::NewStringType::kNormal)
-                            .ToLocalChecked(),
-                   v8::Number::New(isolate, request_time))
-             .FromMaybe(false)) {
-      return;
-    }
+     v8::Local<v8::Object> load_times = v8::Object::New(isolate);

[adamk, 2016/07/20 00:57:22]

Nit: looks like indentation got off somewhere here?

 
-    if (!load_times
-             ->Set(ctx, v8::String::NewFromUtf8(isolate, "startLoadTime",
-                                                v8::NewStringType::kNormal)
-                            .ToLocalChecked(),
-                   v8::Number::New(isolate, start_load_time))
-             .FromMaybe(false)) {
-      return;
-    }
-    if (!load_times
-             ->Set(ctx, v8::String::NewFromUtf8(isolate, "commitLoadTime",
-                                                v8::NewStringType::kNormal)
-                            .ToLocalChecked(),
-                   v8::Number::New(isolate, commit_load_time))
-             .FromMaybe(false)) {
-      return;
-    }
-    if (!load_times
-             ->Set(ctx,
-                   v8::String::NewFromUtf8(isolate, "finishDocumentLoadTime",
-                                           v8::NewStringType::kNormal)
-                       .ToLocalChecked(),
-                   v8::Number::New(isolate, finish_document_load_time))
-             .FromMaybe(false)) {
-      return;
-    }
-    if (!load_times
-             ->Set(ctx, v8::String::NewFromUtf8(isolate, "finishLoadTime",
-                                                v8::NewStringType::kNormal)
-                            .ToLocalChecked(),
-                   v8::Number::New(isolate, finish_load_time))
-             .FromMaybe(false)) {
-      return;
-    }
-    if (!load_times
-             ->Set(ctx, v8::String::NewFromUtf8(isolate, "firstPaintTime",
-                                                v8::NewStringType::kNormal)
-                            .ToLocalChecked(),
-                   v8::Number::New(isolate, first_paint_time))
-             .FromMaybe(false)) {
-      return;
-    }
-    if (!load_times
-             ->Set(ctx,
-                   v8::String::NewFromUtf8(isolate, "firstPaintAfterLoadTime",
-                                           v8::NewStringType::kNormal)
-                       .ToLocalChecked(),
-                   v8::Number::New(isolate, first_paint_after_load_time))
-             .FromMaybe(false)) {
-      return;
-    }
-    if (!load_times
-             ->Set(ctx, v8::String::NewFromUtf8(isolate, "navigationType",
-                                                v8::NewStringType::kNormal)
-                            .ToLocalChecked(),
-                   v8::String::NewFromUtf8(isolate, navigation_type.c_str(),
-                                           v8::NewStringType::kNormal)
-                       .ToLocalChecked())
-             .FromMaybe(false)) {
-      return;
-    }
-    if (!load_times
-             ->Set(ctx, v8::String::NewFromUtf8(isolate, "wasFetchedViaSpdy",
-                                                v8::NewStringType::kNormal)
-                            .ToLocalChecked(),
-                   v8::Boolean::New(isolate, was_fetched_via_spdy))
-             .FromMaybe(false)) {
-      return;
-    }
-    if (!load_times
-             ->Set(ctx, v8::String::NewFromUtf8(isolate, "wasNpnNegotiated",
-                                                v8::NewStringType::kNormal)
-                            .ToLocalChecked(),
-                   v8::Boolean::New(isolate, was_npn_negotiated))
-             .FromMaybe(false)) {
-      return;
-    }
-    if (!load_times
-             ->Set(ctx,
-                   v8::String::NewFromUtf8(isolate, "npnNegotiatedProtocol",
-                                           v8::NewStringType::kNormal)
-                       .ToLocalChecked(),
-                   v8::String::NewFromUtf8(isolate,
-                                           npn_negotiated_protocol.c_str(),
-                                           v8::NewStringType::kNormal)
-                       .ToLocalChecked())
-             .FromMaybe(false)) {
-      return;
-    }
-    if (!load_times
-             ->Set(ctx, v8::String::NewFromUtf8(isolate,
-                                                "wasAlternateProtocolAvailable",
-                                                v8::NewStringType::kNormal)
-                            .ToLocalChecked(),
-                   v8::Boolean::New(isolate, was_alternate_protocol_available))
-             .FromMaybe(false)) {
-      return;
-    }
-    if (!load_times
-             ->Set(ctx, v8::String::NewFromUtf8(isolate, "connectionInfo",
-                                                v8::NewStringType::kNormal)
-                            .ToLocalChecked(),
-                   v8::String::NewFromUtf8(isolate, connection_info.c_str(),
-                                           v8::NewStringType::kNormal)
-                       .ToLocalChecked())
-             .FromMaybe(false)) {
-      return;
-    }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "requestTime", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::Number::New(isolate, request_time))
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "startLoadTime", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::Number::New(isolate, start_load_time))
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "commitLoadTime", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::Number::New(isolate, commit_load_time))
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "finishDocumentLoadTime", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::Number::New(isolate, finish_document_load_time))
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "finishLoadTime", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::Number::New(isolate, finish_load_time))
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "firstPaintTime", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::Number::New(isolate, first_paint_time))
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "firstPaintAfterLoadTime", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::Number::New(isolate,first_paint_after_load_time))
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "navigationType", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::String::NewFromUtf8(isolate, navigation_type.c_str(),
+                                     v8::NewStringType::kNormal)
+             .ToLocalChecked())
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "wasFetchedViaSpdy", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::Boolean::New(isolate, was_fetched_via_spdy))
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "wasNpnNegotiated", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::Boolean::New(isolate, was_npn_negotiated))
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "npnNegotiatedProtocol", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::String::NewFromUtf8(isolate, npn_negotiated_protocol.c_str(),
+                                     v8::NewStringType::kNormal)
+             .ToLocalChecked())
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "wasAlternateProtocolAvailable",
+                 v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::Boolean::New(isolate, was_alternate_protocol_available))
+         .FromMaybe(false)) {
+       return;
+     }
+     if (!load_times->SetAccessor(
+             ctx,
+             v8::String::NewFromUtf8(
+                 isolate, "connectionInfo", v8::NewStringType::kNormal)
+             .ToLocalChecked(),
+             LoadtimesGetter,
+             nullptr,
+             v8::String::NewFromUtf8(isolate, connection_info.c_str(),
+                                     v8::NewStringType::kNormal)
+             .ToLocalChecked())
+         .FromMaybe(false)) {
+       return;
+     }
+
     args.GetReturnValue().Set(load_times);
   }
 
   static void GetCSI(const v8::FunctionCallbackInfo<v8::Value>& args) {
     args.GetReturnValue().SetNull();
     WebLocalFrame* frame = WebLocalFrame::frameForCurrentContext();
     if (!frame) {
       return;
     }
     WebDataSource* data_source = frame->dataSource();
@@ skipping 47 matching lines @@
     }
     args.GetReturnValue().Set(csi);
   }
 };
 
 v8::Extension* LoadTimesExtension::Get() {
   return new LoadTimesExtensionWrapper();
 }
 
 }  // namespace extensions_v8