|Summary:||[CSS Grid Layout] Replace the usage of size_t by unsigned|
|Product:||WebKit||Reporter:||Sergio Villar Senin <svillar>|
|Component:||CSS||Assignee:||Sergio Villar Senin <svillar>|
|Severity:||Normal||CC:||andersca, benjamin, buildbot, darin, jer.noble, kling, rego, rniwa, svillar|
|Version:||528+ (Nightly build)|
|Bug Depends on:||136217|
Description Sergio Villar Senin 2014-08-25 03:29:18 PDT
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.
Comment 1 Sergio Villar Senin 2014-08-25 03:30:21 PDT
See for example: https://bugs.webkit.org/show_bug.cgi?id=135701#c2
Comment 3 Sergio Villar Senin 2014-11-13 07:22:11 PST
Created attachment 241483 [details] Patch Added a missing conversion
Comment 4 Sergio Villar Senin 2014-11-18 00:48:40 PST
Would it be possible to get this reviewed?
Comment 5 Benjamin Poulain 2014-11-18 01:12:49 PST
Comment on attachment 241483 [details] Patch Looks reasonable. If someones creates a grid overflowing unsigned, she/he is crazy :)
Comment 6 Sergio Villar Senin 2014-11-18 01:33:55 PST
(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).