Title: [107685] trunk/Source/WebCore
- Revision
- 107685
- Author
- [email protected]
- Date
- 2012-02-14 01:13:07 -0800 (Tue, 14 Feb 2012)
Log Message
Extend Chromium V8 tracing to cover more cases
https://bugs.webkit.org/show_bug.cgi?id=78507
Patch by Rick Byers <[email protected]> on 2012-02-14
Reviewed by Eric Seidel.
No tests modified because this affects only chrome tracing which we
currently don't try to validate with automated tests.
* bindings/v8/V8EventListener.cpp:
(WebCore::V8EventListener::callListenerFunction):
* bindings/v8/V8Proxy.cpp:
(WebCore::V8Proxy::instrumentedCallFunction):
(WebCore::V8Proxy::newInstance):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (107684 => 107685)
--- trunk/Source/WebCore/ChangeLog 2012-02-14 08:55:38 UTC (rev 107684)
+++ trunk/Source/WebCore/ChangeLog 2012-02-14 09:13:07 UTC (rev 107685)
@@ -1,3 +1,19 @@
+2012-02-14 Rick Byers <[email protected]>
+
+ Extend Chromium V8 tracing to cover more cases
+ https://bugs.webkit.org/show_bug.cgi?id=78507
+
+ Reviewed by Eric Seidel.
+
+ No tests modified because this affects only chrome tracing which we
+ currently don't try to validate with automated tests.
+
+ * bindings/v8/V8EventListener.cpp:
+ (WebCore::V8EventListener::callListenerFunction):
+ * bindings/v8/V8Proxy.cpp:
+ (WebCore::V8Proxy::instrumentedCallFunction):
+ (WebCore::V8Proxy::newInstance):
+
2012-02-14 Pavel Feldman <[email protected]>
Web Inspector: fire stylesheet changed event upon CSS modifications.
Modified: trunk/Source/WebCore/bindings/v8/V8EventListener.cpp (107684 => 107685)
--- trunk/Source/WebCore/bindings/v8/V8EventListener.cpp 2012-02-14 08:55:38 UTC (rev 107684)
+++ trunk/Source/WebCore/bindings/v8/V8EventListener.cpp 2012-02-14 09:13:07 UTC (rev 107685)
@@ -79,10 +79,6 @@
v8::Handle<v8::Value> parameters[1] = { jsEvent };
-#if PLATFORM(CHROMIUM)
- TRACE_EVENT("V8EventListener::callListenerFunction", this, 0);
-#endif
-
if (V8Proxy* proxy = V8Proxy::retrieve(context)) {
Frame* frame = static_cast<Document*>(context)->frame();
if (frame->script()->canExecuteScripts(AboutToExecuteScript))
Modified: trunk/Source/WebCore/bindings/v8/V8Proxy.cpp (107684 => 107685)
--- trunk/Source/WebCore/bindings/v8/V8Proxy.cpp 2012-02-14 08:55:38 UTC (rev 107684)
+++ trunk/Source/WebCore/bindings/v8/V8Proxy.cpp 2012-02-14 09:13:07 UTC (rev 107685)
@@ -428,6 +428,9 @@
v8::Local<v8::Value> result;
{
+#if PLATFORM(CHROMIUM)
+ TRACE_EVENT0("v8", "v8.callFunction");
+#endif
V8RecursionScope recursionScope(frame ? frame->document() : 0);
result = function->Call(receiver, argc, args);
}
@@ -442,6 +445,10 @@
v8::Local<v8::Value> V8Proxy::newInstance(v8::Handle<v8::Function> constructor, int argc, v8::Handle<v8::Value> args[])
{
+#if PLATFORM(CHROMIUM)
+ TRACE_EVENT0("v8", "v8.newInstance");
+#endif
+
// No artificial limitations on the depth of recursion, see comment in
// V8Proxy::callFunction.
v8::Local<v8::Value> result;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
