Ms2ger implemented OpenEndedDictionary! This is what will allow more Headers/Request/Response tests to pass. Woohoo!
So I modified our Headers and Request APIs to adapt to this change. I fought with the borrow checker a lot today, but overall, it was successful. Thanks rustc!
Body
Got dom::Response’s Body text() function to the point that it compiles, but additional tests are not passing. I keep getting the same error Promise is not defined in testharness.js.
Things we learned:
When T doesn’t derive clone, or has a clone implementation, the compiler makes another reference when .clone() is called to a &T, ending up with &&T. When .clone() is called, the compiler derefs &T to T, realizes T cannot be cloned, and so it creates a clone to the reference, hence creating a double reference.
TODO:
Promise in Fetch
Go through failing tests and figure out what I can fix.
Test Curation
Go through the new tests, and categorize them.
OpenEndedDictionary in DOM
Is it time for a PR?
Body
Figure out whether I have a bug. Ask jdm. Then submit PR.
