[RuntimeCallStats] Add more counters

The counters added here are for methods and attributes that take up
a non-trivial portion (4-10%) of time spent doing Blink C++ on some websites
from the v8_top25 benchmark.

Bug: 724543
Change-Id: Idbab1de4b7128a4a934ca22a7b739c2fa5b035bc
Reviewed-on: https://chromium-review.googlesource.com/565682
Commit-Queue: Adithya Srinivasan <adithyas@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#486143}

third_party/WebKit/Source/bindings/core/v8/V8EventListenerHelper.cpp

@@ -41,6 +41,8 @@ EventListener* V8EventListenerHelper::GetEventListener(
     v8::Local<v8::Value> value,
     bool is_attribute,
     ListenerLookupType lookup) {
+  RUNTIME_CALL_TIMER_SCOPE(script_state->GetIsolate(),
+                           RuntimeCallStats::CounterId::kGetEventListener);
   if (lookup == kListenerFindOnly) {
     // Used by EventTarget::removeEventListener, specifically
     // EventTargetV8Internal::removeEventListenerMethod

third_party/WebKit/Source/core/dom/Document.idl

@@ -95,7 +95,7 @@ typedef (HTMLScriptElement or SVGScriptElement) HTMLOrSVGScriptElement;
     [PutForwards=href, Unforgeable] readonly attribute Location? location;
     [RaisesException=Setter] attribute DOMString domain;
     readonly attribute DOMString referrer;
-    [RaisesException] attribute DOMString cookie;
+    [RaisesException, RuntimeCallStatsCounter=DocumentCookie] attribute DOMString cookie;
     readonly attribute DOMString lastModified;
     readonly attribute DocumentReadyState readyState;
 

third_party/WebKit/Source/core/dom/DocumentFragment.cpp

@@ -25,6 +25,8 @@
 #include "core/dom/Document.h"
 #include "core/html/parser/HTMLDocumentParser.h"
 #include "core/xml/parser/XMLDocumentParser.h"
+#include "platform/bindings/RuntimeCallStats.h"
+#include "platform/bindings/V8PerIsolateData.h"
 
 namespace blink {
 
@@ -67,6 +69,9 @@ Node* DocumentFragment::cloneNode(bool deep, ExceptionState&) {
 void DocumentFragment::ParseHTML(const String& source,
                                  Element* context_element,
                                  ParserContentPolicy parser_content_policy) {
+  RUNTIME_CALL_TIMER_SCOPE(
+      V8PerIsolateData::MainThreadIsolate(),
+      RuntimeCallStats::CounterId::kDocumentFragmentParseHTML);
   HTMLDocumentParser::ParseDocumentFragment(source, this, context_element,
                                             parser_content_policy);
 }

third_party/WebKit/Source/core/dom/Element.idl

@@ -91,7 +91,7 @@ interface Element : Node {
     // CSSOM View Module
     // https://dev.w3.org/csswg/cssom-view/#extension-to-the-element-interface
     DOMRectList getClientRects();
-    DOMRect getBoundingClientRect();
+    [RuntimeCallStatsCounter=ElementGetBoundingClientRect] DOMRect getBoundingClientRect();
 
     // TODO(sunyunjia): Add default value for scrollIntoView() once
     // crbug.com/734599 is fixed.

third_party/WebKit/Source/core/dom/Node.idl

@@ -75,7 +75,7 @@
     boolean isDefaultNamespace(DOMString? namespaceURI);
 
     [CEReactions, CustomElementCallbacks, PerWorldBindings, RaisesException] Node insertBefore(Node node, Node? child);
-    [CEReactions, CustomElementCallbacks, PerWorldBindings, RaisesException] Node appendChild(Node node);
+    [CEReactions, CustomElementCallbacks, PerWorldBindings, RaisesException, RuntimeCallStatsCounter=NodeAppendChild] Node appendChild(Node node);
     [CEReactions, CustomElementCallbacks, PerWorldBindings, RaisesException] Node replaceChild(Node node, Node child);
-    [CEReactions, CustomElementCallbacks, RaisesException] Node removeChild(Node child);
+    [CEReactions, CustomElementCallbacks, RaisesException, RuntimeCallStatsCounter=NodeRemoveChild] Node removeChild(Node child);
 };

third_party/WebKit/Source/platform/bindings/RuntimeCallStats.h

@@ -176,20 +176,28 @@ class PLATFORM_EXPORT RuntimeCallStats {
   V(AssociateObjectWithWrapper)                                         \
   V(CollectGarbage)                                                     \
   V(CreateWrapper)                                                      \
+  V(GetEventListener)                                                   \
+  V(HasInstance)                                                        \
+  V(DocumentFragmentParseHTML)                                          \
   V(PaintContents)                                                      \
   V(PerformIdleLazySweep)                                               \
   V(ProcessStyleSheet)                                                  \
   V(ToExecutionContext)                                                 \
   V(ToV8DOMWindow)                                                      \
+  V(ToV8SequenceInternal)                                               \
   V(UpdateLayerPositionsAfterLayout)                                    \
   V(UpdateLayout)                                                       \
   V(UpdateStyle)                                                        \
   V(SetReturnValueFromStringSlow)                                       \
   V(V8ExternalStringSlow)                                               \
   V(V8)                                                                 \
+  BINDINGS_METHOD(V, ElementGetBoundingClientRect)                      \
   BINDINGS_METHOD(V, EventTargetDispatchEvent)                          \
   BINDINGS_METHOD(V, HTMLElementClick)                                  \
+  BINDINGS_METHOD(V, NodeAppendChild)                                   \
+  BINDINGS_METHOD(V, NodeRemoveChild)                                   \
   BINDINGS_METHOD(V, WindowSetTimeout)                                  \
+  BINDINGS_ATTRIBUTE(V, DocumentCookie)                                 \
   BINDINGS_ATTRIBUTE(V, ElementInnerHTML)                               \
   V(TestCounter1)                                                       \
   V(TestCounter2)                                                       \

third_party/WebKit/Source/platform/bindings/ToV8.h

@@ -233,6 +233,8 @@ inline v8::Local<v8::Value> ToV8SequenceInternal(
     const Sequence& sequence,
     v8::Local<v8::Object> creation_context,
     v8::Isolate* isolate) {
+  RUNTIME_CALL_TIMER_SCOPE(isolate,
+                           RuntimeCallStats::CounterId::kToV8SequenceInternal);
   v8::Local<v8::Array> array;
   {
     v8::Context::Scope context_scope(creation_context->CreationContext());

third_party/WebKit/Source/platform/bindings/V8PerIsolateData.cpp

@@ -221,6 +221,8 @@ void V8PerIsolateData::ClearScriptRegexpContext() {
 bool V8PerIsolateData::HasInstance(
     const WrapperTypeInfo* untrusted_wrapper_type_info,
     v8::Local<v8::Value> value) {
+  RUNTIME_CALL_TIMER_SCOPE(GetIsolate(),
+                           RuntimeCallStats::CounterId::kHasInstance);
   return HasInstance(untrusted_wrapper_type_info, value,
                      interface_template_map_for_main_world_) ||
          HasInstance(untrusted_wrapper_type_info, value,