Implement HTMLTableElement#createTBody

[dzbarsky]

[Ms2ger]

Thank you for your PR.

-S-awaiting-review +S-needs-code-changes

---

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, 4 unresolved discussions, all commit checks successful.

---

^{components/script/dom/htmltableelement.rs, line 92 [r1] (raw file):}
Feel free to make this return HTMLTableSectionElement, even though that's not what the spec says.

---

^{components/script/dom/htmltableelement.rs, line 101 [r1] (raw file):}
Indentation is weird; maybe brace the closure.

---

^{components/script/dom/htmltableelement.rs, line 103 [r1] (raw file):}
Maybe use and_then.

---

^{components/script/dom/htmltableelement.rs, line 106 [r1] (raw file):}
Just r() (on RootedReference).

---

Comments from the review on Reviewable.io

[dzbarsky]

Review status: 2 of 4 files reviewed at latest revision, 4 unresolved discussions, some commit checks pending.

---

^{components/script/dom/htmltableelement.rs, line 106 [r1] (raw file):}
Oh I thought that was gone after smup. Looks like it's back or something.

---

Comments from the review on Reviewable.io

[Ms2ger]

-S-awaiting-review +S-awaiting-merge

@bors-servo r+

---

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, 3 unresolved discussions, all commit checks successful.

---

Comments from the review on Reviewable.io

[bors-servo]

📌 Commit 992a1e1 has been approved by Ms2ger

[bors-servo]

⌛ Testing commit 992a1e1 with merge 454b2a7...

[bors-servo]

💔 Test failed - linux2

[Ms2ger]

/html/dom/interfaces.html
-------------------------
PASS expected FAIL HTMLTableElement interface: operation createTBody()
PASS expected FAIL HTMLTableElement interface: document.createElement("table") must inherit property "createTBody" with the proper type (10)

[bors-servo]

☔ The latest upstream changes (presumably #7063) made this pull request unmergeable. Please resolve the merge conflicts.

[Ms2ger]

@bors-servo r+

[bors-servo]

📌 Commit 54adae2 has been approved by Ms2ger

[bors-servo]

⌛ Testing commit 54adae2 with merge 44d93bc...

[bors-servo]

☀️ Test successful - android, gonk, linux1, linux2, mac1, mac2, mac3