Changeset 206408 in webkit for trunk/Source/JavaScriptCore/profiler/ProfilerDatabase.cpp

Timestamp:

Sep 26, 2016, 7:46:19 PM (9 years ago)

Author:

[email protected]

Message:

Unreviewed, rolling out r206405.
​https://bugs.webkit.org/show_bug.cgi?id=162588

This change caused LayoutTest crashes. (Requested by
ryanhaddad on #webkit).

Reverted changeset:

"Add some needed CatchScopes in code that should not throw."
​https://bugs.webkit.org/show_bug.cgi?id=162584
​http://trac.webkit.org/changeset/206405

File:

• trunk/Source/JavaScriptCore/profiler/ProfilerDatabase.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/profiler/ProfilerDatabase.cpp

@@ -135 +135 @@
 bool Database::save(const char* filename) const
 {
-    auto scope = DECLARE_CATCH_SCOPE(m_vm);
     auto out = FilePrintStream::open(filename, "w");
     if (!out)
         return false;
     
-    String data = toJSON();
-    if (UNLIKELY(scope.exception())) {
-        scope.clearException();
-        return false;
-    }
-    out->print(data);
+    out->print(toJSON());
     return true;
 }