Title: [154892] trunk/Source/_javascript_Core
- Revision
- 154892
- Author
- [email protected]
- Date
- 2013-08-30 10:02:48 -0700 (Fri, 30 Aug 2013)
Log Message
Cleaning errorDescriptionForValue after r154839
https://bugs.webkit.org/show_bug.cgi?id=120531
Patch by Chris Curtis <[email protected]> on 2013-08-30
Reviewed by Darin Adler.
Changed the assert to ASSERT_NOT_REACHED, now that r154839 has landed. errorDescriptionForValue
can assert again that the parameterized JSValue is !isEmpty().
* runtime/ExceptionHelpers.cpp:
(JSC::errorDescriptionForValue):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (154891 => 154892)
--- trunk/Source/_javascript_Core/ChangeLog 2013-08-30 16:45:58 UTC (rev 154891)
+++ trunk/Source/_javascript_Core/ChangeLog 2013-08-30 17:02:48 UTC (rev 154892)
@@ -1,3 +1,16 @@
+2013-08-30 Chris Curtis <[email protected]>
+
+ Cleaning errorDescriptionForValue after r154839
+ https://bugs.webkit.org/show_bug.cgi?id=120531
+
+ Reviewed by Darin Adler.
+
+ Changed the assert to ASSERT_NOT_REACHED, now that r154839 has landed. errorDescriptionForValue
+ can assert again that the parameterized JSValue is !isEmpty().
+
+ * runtime/ExceptionHelpers.cpp:
+ (JSC::errorDescriptionForValue):
+
2013-08-30 Antti Koivisto <[email protected]>
Remove code behind ENABLE(DIALOG_ELEMENT)
Modified: trunk/Source/_javascript_Core/runtime/ExceptionHelpers.cpp (154891 => 154892)
--- trunk/Source/_javascript_Core/runtime/ExceptionHelpers.cpp 2013-08-30 16:45:58 UTC (rev 154891)
+++ trunk/Source/_javascript_Core/runtime/ExceptionHelpers.cpp 2013-08-30 17:02:48 UTC (rev 154892)
@@ -109,8 +109,8 @@
return jsString(exec, object->methodTable()->className(object));
}
- ASSERT(v.isEmpty());
- // FIXME: https://bugs.webkit.org/show_bug.cgi?id=120080 The JSValue should never be empty in this function.
+ // The JSValue should never be empty, so this point in the code should never be reached.
+ ASSERT_NOT_REACHED();
return vm.smallStrings.emptyString();
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
