[webkit-reviews] review granted: [Bug 6909] Replace QValueVector and QPtrVector with Vector : [Attachment 6069] vectorize

[bugzilla-request-daemon at opendarwin.org]

Darin Adler <darin at apple.com> has granted Maciej Stachowiak <mjs at apple.com>'s
request for review:
Bug 6909: Replace QValueVector and QPtrVector with Vector
http://bugzilla.opendarwin.org/show_bug.cgi?id=6909

Attachment 6069: vectorize
http://bugzilla.opendarwin.org/attachment.cgi?id=6069&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
For the m_negZOrderList and m_posZOrderList, I noticed that the fetch of the
cout went inside the loop. In general, is there a reason to use iterators to
walk a vector? To me it seems easier to understand a loop that uses numeric
indices.

Love the use of "256" in jobsToCancel. A nice optimization to have at our
fingertips!

r=me