Title: [88166] trunk/Source/WebCore
Revision
88166
Author
[email protected]
Date
2011-06-06 07:15:30 -0700 (Mon, 06 Jun 2011)

Log Message

2011-06-06  Steve Block  <[email protected]>

        Reviewed by Yury Semikhatsky.

        Use of ENABLE(INSPECTOR) is inconsistent
        https://bugs.webkit.org/show_bug.cgi?id=62118

        Correctly guard implementation of ScriptProfiler.

        No new tests, build fix only.

        * bindings/v8/ScriptProfiler.cpp:
        (WebCore::ScriptProfiler::initialize):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (88165 => 88166)


--- trunk/Source/WebCore/ChangeLog	2011-06-06 14:12:11 UTC (rev 88165)
+++ trunk/Source/WebCore/ChangeLog	2011-06-06 14:15:30 UTC (rev 88166)
@@ -1,3 +1,17 @@
+2011-06-06  Steve Block  <[email protected]>
+
+        Reviewed by Yury Semikhatsky.
+
+        Use of ENABLE(INSPECTOR) is inconsistent
+        https://bugs.webkit.org/show_bug.cgi?id=62118
+
+        Correctly guard implementation of ScriptProfiler.
+
+        No new tests, build fix only.
+
+        * bindings/v8/ScriptProfiler.cpp:
+        (WebCore::ScriptProfiler::initialize):
+
 2011-06-04  Dominic Cooney  <[email protected]>
 
         Reviewed by Dimitri Glazkov.

Modified: trunk/Source/WebCore/bindings/v8/ScriptProfiler.cpp (88165 => 88166)


--- trunk/Source/WebCore/bindings/v8/ScriptProfiler.cpp	2011-06-06 14:12:11 UTC (rev 88165)
+++ trunk/Source/WebCore/bindings/v8/ScriptProfiler.cpp	2011-06-06 14:15:30 UTC (rev 88166)
@@ -40,6 +40,7 @@
 
 namespace WebCore {
 
+#if ENABLE(INSPECTOR)
 void ScriptProfiler::start(ScriptState* state, const String& title)
 {
     v8::HandleScope hs;
@@ -108,10 +109,13 @@
     Node* node = V8Node::toNative(wrapper.As<v8::Object>());
     return node ? new RetainedDOMInfo(node) : 0;
 }
+#endif // ENABLE(INSPECTOR)
 
 void ScriptProfiler::initialize()
 {
+#if ENABLE(INSPECTOR)
     v8::HeapProfiler::DefineWrapperClass(v8DOMSubtreeClassId, &retainedDOMInfo);
+#endif // ENABLE(INSPECTOR)
 }
 
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to