Add validation to length, packets and packetLength parameters

Add validation to length, packets and packet_length parameters
of the Usb Api. Parameter limits are determined from the following:

packets: 4MB from http://msdn.microsoft.com/en-us/library/windows/hardware/ff538112(v=vs.85).aspx
packet_length: 65k since wMaxPacketSize is 16bit integer: http://msdn.microsoft.com/en-us/library/ee437177.aspx

BUG=169765
R=gdk@chromium.org,miket@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182128

[miket_OOO, 2013-01-28 21:23:40]

lgtm

https://codereview.chromium.org/12096024/diff/1/chrome/browser/extensions/api...
File chrome/browser/extensions/api/usb/usb_api.cc (right):

https://codereview.chromium.org/12096024/diff/1/chrome/browser/extensions/api...
chrome/browser/extensions/api/usb/usb_api.cc:646: unsigned int packets =
transfer.packets;
Would a size_t work here?

[meacer, 2013-01-28 21:26:25]

https://codereview.chromium.org/12096024/diff/1/chrome/browser/extensions/api...
File chrome/browser/extensions/api/usb/usb_api.cc (right):

https://codereview.chromium.org/12096024/diff/1/chrome/browser/extensions/api...
chrome/browser/extensions/api/usb/usb_api.cc:646: unsigned int packets =
transfer.packets;
On 2013/01/28 21:23:40, miket wrote:
> Would a size_t work here?

I made it an int just because it's used in a multiplication below.

[commit-bot: I haz the power, 2013-01-28 23:46:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/meacer@chromium.org/12096024/1

[commit-bot: I haz the power, 2013-01-28 23:46:15]

Presubmit check for 12096024-1 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/browser/usb

[Mustafa Acer, 2013-01-28 23:48:31]

+bryeung for OWNERS review

[Garret Kelly, 2013-01-29 14:48:00]

LGTM

[Garret Kelly, 2013-01-29 14:48:44]

Sorry, I thought I was on this for informational purposes.

[bryeung, 2013-01-29 14:54:56]

lgtm

[commit-bot: I haz the power, 2013-01-29 18:01:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/meacer@chromium.org/12096024/1

[commit-bot: I haz the power, 2013-01-29 18:49:49]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-02-13 00:52:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/meacer@chromium.org/12096024/15001

[commit-bot: I haz the power, 2013-02-13 06:06:08]

Change committed as 182128