Get rid of custom code for Document::createTouchList()

Source/core/dom/Document.h

Index: Source/core/dom/Document.h
diff --git a/Source/core/dom/Document.h b/Source/core/dom/Document.h
index 9eb2cdade25da88eefbeec1c447afefaf3000076..e69a7e2b3a586bfa5715942e722b6d81ee937263 100644
--- a/Source/core/dom/Document.h
+++ b/Source/core/dom/Document.h
@@ -972,6 +972,7 @@ public:
     bool isDelayingLoadEvent() const { return m_loadEventDelayCount; }
 
     PassRefPtr<Touch> createTouch(DOMWindow*, EventTarget*, int identifier, int pageX, int pageY, int screenX, int screenY, int radiusX, int radiusY, float rotationAngle, float force) const;
+    PassRefPtr<TouchList> createTouchList(Vector<RefPtr<Touch> >&) const;

[arv (Not doing code reviews), 2013/09/16 13:06:20]

Shouldn't this be const Vector& since createTouchList should not change the
vector?

[do-not-use, 2013/09/16 13:25:27]

Why shouldn't createTouchList() modify the Vector? The caller (generated
bindings) constructs the Vector especially for Document and does not use it
afterwards.

This reference is not const on purpose to avoid copying the Touch vector when
constructing the TouchList (I use Vector::swap()). If I don't do that then the
generated code will be slower than the custom one due the to extra Vector copy.

 
     const DocumentTiming* timing() const { return &m_documentTiming; }