GitHub PushEvent at 2014-09-12 07:40:15
Repository: Blink
Actor: yoavweiss
Payload: {"shas":[["74ec9a9cc07b1dd5f7daf85980e05405e3d18ca9","9f0d208cf30f99c24f65de1bc3fbe5d25526ac82@opera.com","Use conversion helpers in V8Binding.cpp for [Clamp] method arguments\n\nThere was custom generated code specifically for handling arguments with\n[Clamp], which is quite unnecessary since the toUInt*() have an integer\nconversion configuration parameter that among other things can be used to\nrequest clamping.\n\nRelated: Fix the somewhat broken handling the |Clamp| integer conversion\nconfiguration in the toUInt*() helpers. An optimization code path for the\ncase that the V8 value is an int32 forgot to do clamping, there was no\nclamping at all in toUInt64(), and +/- Infinity was converted to zero\ninstead of being clamped.\n\nClamping support is not added to toInt64(), since the clampTo<> helper in\nSource/wtf/MathExtras.h explicitly doesn't support |long long int|, which\nis int64_t on some platforms.\n\nBUG=413257\n\nReview URL: https://codereview.chromium.org/563793002\n\ngit-svn-id: svn://svn.chromium.org/blink/trunk@181882 bbb929c8-8fbe-4397-9dbb-9b2b20218538","jl@opera.com",true]],"size":1,"ref":"refs/heads/master","head":"74ec9a9cc07b1dd5f7daf85980e05405e3d18ca9"}