Bug 1920955 - Part 48: Simplify ToTemporalDuration. r=sfink

<tc39/proposal-temporal#2943> should address the fixme note.

Differential Revision: https://phabricator.services.mozilla.com/D223554

UltraBlame original commit: de7fc134aa463dddcc4f75ff9ef2ba2b9b670a6f

Author: marco-c

js/src/builtin/temporal/Duration.cpp

@@ -1154,38 +1154,12 @@ bool js::temporal::ToTemporalDurationRecord(JSContext* cx,
 
 
 
-Wrapped<DurationObject*> js::temporal::ToTemporalDuration(JSContext* cx,
-                                                          Handle<Value> item) {
+static bool ToTemporalDuration(JSContext* cx, Handle<Value> item,
+                               Duration* result) {
 
-  if (item.isObject()) {
-    JSObject* itemObj = &item.toObject();
-    if (itemObj->canUnwrapAs<DurationObject>()) {
-      return itemObj;
-    }
-  }
-
-  
-  Duration result;
-  if (!ToTemporalDurationRecord(cx, item, &result)) {
-    return nullptr;
-  }
 
 
-  return CreateTemporalDuration(cx, result);
-}
-
-
-
-
-bool js::temporal::ToTemporalDuration(JSContext* cx, Handle<Value> item,
-                                      Duration* result) {
-  auto obj = ToTemporalDuration(cx, item);
-  if (!obj) {
-    return false;
-  }
-
-  *result = ToDuration(&obj.unwrap());
-  return true;
+  return ToTemporalDurationRecord(cx, item, result);
 }
 
 
@@ -3366,28 +3340,18 @@ static bool DurationConstructor(JSContext* cx, unsigned argc, Value* vp) {
 static bool Duration_from(JSContext* cx, unsigned argc, Value* vp) {
   CallArgs args = CallArgsFromVp(argc, vp);
 
-  Handle<Value> item = args.get(0);
-
 
-  if (item.isObject()) {
-    if (auto* duration = item.toObject().maybeUnwrapIf<DurationObject>()) {
-      auto* result = CreateTemporalDuration(cx, ToDuration(duration));
-      if (!result) {
-        return false;
-      }
-
-      args.rval().setObject(*result);
-      return true;
-    }
+  Duration result;
+  if (!ToTemporalDuration(cx, args.get(0), &result)) {
+    return false;
   }
 
-  
-  auto result = ToTemporalDuration(cx, item);
-  if (!result) {
+  auto* obj = CreateTemporalDuration(cx, result);
+  if (!obj) {
     return false;
   }
 
-  args.rval().setObject(*result);
+  args.rval().setObject(*obj);
   return true;
 }
 

js/src/builtin/temporal/Duration.h

@@ -264,18 +264,6 @@ DurationObject* CreateTemporalDuration(JSContext* cx, const Duration& duration);
 
 
 
-Wrapped<DurationObject*> ToTemporalDuration(JSContext* cx,
-                                            JS::Handle<JS::Value> item);
-
-
-
-
-bool ToTemporalDuration(JSContext* cx, JS::Handle<JS::Value> item,
-                        Duration* result);
-
-
-
-
 bool ToTemporalDurationRecord(JSContext* cx,
                               JS::Handle<JS::Value> temporalDurationLike,
                               Duration* result);