Changeset 206658 in webkit
- Timestamp:
- Sep 30, 2016 12:47:54 PM (8 years ago)
- Location:
- trunk/Source/JavaScriptCore
- Files:
-
- 2 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/JavaScriptCore/ChangeLog
r206654 r206658 1 2016-09-30 Mark Lam <mark.lam@apple.com> 2 3 Remove the dumping of the stack back trace in VM::verifyExceptionCheckNeedIsSatisfied(). 4 https://bugs.webkit.org/show_bug.cgi?id=162797 5 6 Reviewed by Geoffrey Garen. 7 8 This is because the RELEASE_ASSERT() that follows immediately after will also 9 dump the stack back trace. Hence, the first dump will be redundant. 10 11 Also removed an extra space in the dataLog output. 12 13 * runtime/VM.cpp: 14 (JSC::VM::verifyExceptionCheckNeedIsSatisfied): 15 1 16 2016-09-30 Joseph Pecoraro <pecoraro@apple.com> 2 17 -
trunk/Source/JavaScriptCore/runtime/VM.cpp
r206267 r206658 907 907 "ERROR: Unchecked JS exception:\n" 908 908 " This scope can throw a JS exception: ", throwLocation, "\n" 909 " (ExceptionScope::m_recursionDepth was 909 " (ExceptionScope::m_recursionDepth was ", throwDepth, ")\n" 910 910 " But the exception was unchecked as of this scope: ", location, "\n" 911 " (ExceptionScope::m_recursionDepth was 911 " (ExceptionScope::m_recursionDepth was ", recursionDepth, ")\n" 912 912 "\n"); 913 WTFReportBacktrace();914 913 915 914 RELEASE_ASSERT(!m_needExceptionCheck);
Note: See TracChangeset
for help on using the changeset viewer.