Editorial: Move IsValidDuration check into ToTemporalDurationRecord

This does not change anything observable because directly after all call
sites of ToTemporalDurationRecord, IsValidDuration was checked anyway.
However, this way is conceptually easier because it implements the
invariant that no Duration Record can be created with invalid values in
it.

Author: ptomato

spec/duration.html

@@ -920,7 +920,7 @@ <h1>
         1. Else,
           1. Let _string_ be ? ToString(_item_).
           1. Let _result_ be ? ParseTemporalDurationString(_string_).
-        1. Return ? CreateTemporalDuration(_result_.[[Years]], _result_.[[Months]], _result_.[[Weeks]], _result_.[[Days]], _result_.[[Hours]], _result_.[[Minutes]], _result_.[[Seconds]], _result_.[[Milliseconds]], _result_.[[Microseconds]], _result_.[[Nanoseconds]]).
+        1. Return ! CreateTemporalDuration(_result_.[[Years]], _result_.[[Months]], _result_.[[Weeks]], _result_.[[Days]], _result_.[[Hours]], _result_.[[Minutes]], _result_.[[Seconds]], _result_.[[Milliseconds]], _result_.[[Microseconds]], _result_.[[Nanoseconds]]).
       </emu-alg>
     </emu-clause>
 
@@ -950,6 +950,8 @@ <h1>
             1. Set _result_'s field whose name is the Field Name value of the current row to _val_.
         1. If _any_ is *false*, then
           1. Throw a *TypeError* exception.
+        1. If ! IsValidDuration(_result_.[[Years]], _result_.[[Months]], _result_.[[Weeks]] _result_.[[Days]], _result_.[[Hours]], _result_.[[Minutes]], _result_.[[Seconds]], _result_.[[Milliseconds]], _result_.[[Microseconds]], _result_.[[Nanoseconds]]) is *false*, then
+          1. Throw a *RangeError* exception.
         1. Return _result_.
       </emu-alg>
     </emu-clause>
@@ -1787,7 +1789,6 @@ <h1>
           1. Let _duration_ be ? ParseTemporalDurationString(_str_).
         1. Else,
           1. Let _duration_ be ? ToTemporalDurationRecord(_temporalDurationLike_).
-        1. If ! IsValidDuration(_duration_.[[Years]], _duration_.[[Months]], _duration_.[[Weeks]], _duration_.[[Days]], _duration_.[[Hours]], _duration_.[[Minutes]], _duration_.[[Seconds]], _duration_.[[Milliseconds]], _duration_.[[Microseconds]], _duration_.[[Nanoseconds]]) is *false*, throw a *RangeError* exception.
         1. For each row of <emu-xref href="#table-temporal-duration-record-fields"></emu-xref>, except the header row, in table order, do
           1. Let _prop_ be the Property Name value of the current row.
           1. Let _value_ be _duration_'s field whose name is the Field Name value of the current row.