size_t is generally used for sizes of memory objects. We're using it for indexes and spans in the grid. The fact that we're using Vectors to represent the grid should not force us to use size_t, unsigned seems to fit better in this case.
See for example: https://bugs.webkit.org/show_bug.cgi?id=135701#c2
Created attachment 241434 [details] Patch
Created attachment 241483 [details] Patch Added a missing conversion
Would it be possible to get this reviewed?
Comment on attachment 241483 [details] Patch Looks reasonable. If someones creates a grid overflowing unsigned, she/he is crazy :)
(In reply to comment #5) > Comment on attachment 241483 [details] > Patch > > Looks reasonable. > If someones creates a grid overflowing unsigned, she/he is crazy :) Yeah :) In any case, I have uploaded this only after landing bug 136217 that ensures that we won't overflow unsigned as it sets a hard limit to the number of tracks (1 million as recommended by specs).
Committed r176390: <http://trac.webkit.org/changeset/176390>