Spread resolved data on updateWith()

[marcoscaceres]

This builds on #668 to allow incremental data updates as proposed in #649.

---

Preview | Diff

[marcoscaceres]

@domenic, the interesting stuff only happens in fc5ea16. The rest is just #668.

[domenic]

So it's better to stay at the Infra level than to drop down to the ECMAScript level. Otherwise what happens here will be different if someone monkeys with Object.prototype, for example. You'll need to define some kind of ordered map merging operation, or let us do that for you in Infra.

BUT, as I pointed out in #649 (comment), I don't think this patch is necessary at all. Let's continue discussing there.

[marcoscaceres]

Confirmed, not needed. Sorry about the noise. Someone needs to fire the person writing test tho :)

[marcoscaceres]

@domenic, I'm still kinda wondering if we should add "property spread" to infra (even if we didn't need it here).

Consider, after validation the values object, it would be nice to just spread them over request.[[details]] instead of adding/replacing the properties one by one.

[domenic]

Yeah, I definitely think some kind of "merge-with-overwrite" for ordered maps would be reasonable. I'd prefer to wait until we have a consumer, but we could open a tracking issue perhaps.