Title: [103004] trunk/Source/WebCore
- Revision
- 103004
- Author
- [email protected]
- Date
- 2011-12-15 17:17:45 -0800 (Thu, 15 Dec 2011)
Log Message
[Chromium] Avoid strdup() for extra argument when tracing is disabled.
https://bugs.webkit.org/show_bug.cgi?id=74637
Patch by Daniel Sievers <[email protected]> on 2011-12-15
Reviewed by James Robinson.
* platform/chromium/TraceEvent.h:
(WebCore::internal::ScopeTracer::ScopeTracer):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (103003 => 103004)
--- trunk/Source/WebCore/ChangeLog 2011-12-16 01:12:45 UTC (rev 103003)
+++ trunk/Source/WebCore/ChangeLog 2011-12-16 01:17:45 UTC (rev 103004)
@@ -1,3 +1,13 @@
+2011-12-15 Daniel Sievers <[email protected]>
+
+ [Chromium] Avoid strdup() for extra argument when tracing is disabled.
+ https://bugs.webkit.org/show_bug.cgi?id=74637
+
+ Reviewed by James Robinson.
+
+ * platform/chromium/TraceEvent.h:
+ (WebCore::internal::ScopeTracer::ScopeTracer):
+
2011-12-15 Rafael Weinstein <[email protected]>
[MutationObservers] Add a document-level flag that can trivially be checked to avoid doing unnessary work if mutation observers absent
Modified: trunk/Source/WebCore/platform/chromium/TraceEvent.h (103003 => 103004)
--- trunk/Source/WebCore/platform/chromium/TraceEvent.h 2011-12-16 01:12:45 UTC (rev 103003)
+++ trunk/Source/WebCore/platform/chromium/TraceEvent.h 2011-12-16 01:17:45 UTC (rev 103004)
@@ -59,7 +59,7 @@
, m_id(id)
{
PlatformSupport::traceEventBegin(name, id, extra); \
- if (extra)
+ if (extra && PlatformSupport::isTraceEventEnabled())
m_extra = adoptArrayPtr(strdup(extra));
}
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
