Title: [96869] trunk/Source/_javascript_Core
- Revision
- 96869
- Author
- gav...@chromium.org
- Date
- 2011-10-06 16:02:05 -0700 (Thu, 06 Oct 2011)
Log Message
REGRESSION (r96595): First frame in assertion backtraces is no longer labeled "1"
https://bugs.webkit.org/show_bug.cgi?id=69556
Reviewed by Adam Roben.
* wtf/Assertions.cpp:
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (96868 => 96869)
--- trunk/Source/_javascript_Core/ChangeLog 2011-10-06 22:55:20 UTC (rev 96868)
+++ trunk/Source/_javascript_Core/ChangeLog 2011-10-06 23:02:05 UTC (rev 96869)
@@ -1,3 +1,12 @@
+2011-10-06 Gavin Peters <gav...@chromium.org>
+
+ REGRESSION (r96595): First frame in assertion backtraces is no longer labeled "1"
+ https://bugs.webkit.org/show_bug.cgi?id=69556
+
+ Reviewed by Adam Roben.
+
+ * wtf/Assertions.cpp:
+
2011-10-06 Filip Pizlo <fpi...@apple.com>
DFG implementation of UInt32ToNumber is missing a break statement
Modified: trunk/Source/_javascript_Core/wtf/Assertions.cpp (96868 => 96869)
--- trunk/Source/_javascript_Core/wtf/Assertions.cpp 2011-10-06 22:55:20 UTC (rev 96868)
+++ trunk/Source/_javascript_Core/wtf/Assertions.cpp 2011-10-06 23:02:05 UTC (rev 96869)
@@ -211,10 +211,11 @@
if (mangledName)
cxaDemangled = abi::__cxa_demangle(mangledName, 0, 0, 0);
#endif
+ const int frameNumber = i - framesToSkip + 1;
if (mangledName || cxaDemangled)
- fprintf(stderr, "%-3d %p %s\n", i, samples[i], cxaDemangled ? cxaDemangled : mangledName);
+ fprintf(stderr, "%-3d %p %s\n", frameNumber, samples[i], cxaDemangled ? cxaDemangled : mangledName);
else
- fprintf(stderr, "%-3d %p\n", i, samples[i]);
+ fprintf(stderr, "%-3d %p\n", frameNumber, samples[i]);
free(cxaDemangled);
}
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes