Remove conversion of non-string literals and remove From<String> for …

…JsValue

core/engine/src/error.rs

@@ -14,8 +14,7 @@ use boa_gc::{custom_trace, Finalize, Trace};
 use boa_macros::js_str;
 use thiserror::Error;
 
-/// Create an opaque error object from a value. This can be a literal or any
-/// value that can be converted to a `JsValue`.
+/// Create an opaque error object from a value or string literal.
 ///
 /// Can be used with an expression that converts into `JsValue` or a format
 /// string with arguments.
@@ -34,11 +33,17 @@ use thiserror::Error;
 /// let error = js_error!("error: {}", 5);
 /// assert_eq!(error.as_opaque().unwrap().to_string(context).unwrap(), "error: 5");
 ///
-/// let error = js_error!(true);
+/// // Non-string literals must be used as an expression.
+/// let error = js_error!({ true });
 /// assert_eq!(error.as_opaque().unwrap(), &JsValue::from(true));
 /// ```
 #[macro_export]
 macro_rules! js_error {
+    ($value: literal) => {
+        $crate::JsError::from_opaque($crate::JsValue::from(
+            $crate::js_string!($value)
+        ))
+    };
     ($value: expr) => {
         $crate::JsError::from_opaque(
             $crate::JsValue::from($value)

core/engine/src/value/conversions/mod.rs

@@ -39,24 +39,6 @@ impl From<char> for JsValue {
     }
 }
 
-impl From<&str> for JsValue {
-    #[inline]
-    fn from(value: &str) -> Self {
-        let _timer = Profiler::global().start_event("From<&str>", "value");
-
-        Self::from(js_string!(value))
-    }
-}
-
-impl From<String> for JsValue {
-    #[inline]
-    fn from(value: String) -> Self {
-        let _timer = Profiler::global().start_event("From<&str>", "value");
-
-        Self::from(JsString::from(value))
-    }
-}
-
 impl From<JsSymbol> for JsValue {
     #[inline]
     fn from(value: JsSymbol) -> Self {