Added bounds checks to the URLLoader prefetch buffer threshhold.

Added bounds checks to the URLLoader prefetch buffer threshhold.


BUG=89842
TEST=./ui_tests --gtest_filter=PPAPITest.URLLoader*PrefetchBufferThreshold


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=117970

[dmichael (off chromium), 2012-01-13 17:46:02]

lgtm

http://codereview.chromium.org/9139076/diff/6/ppapi/tests/test_url_loader.cc
File ppapi/tests/test_url_loader.cc (right):

http://codereview.chromium.org/9139076/diff/6/ppapi/tests/test_url_loader.cc#...
ppapi/tests/test_url_loader.cc:914: 
I think it would make sense to collapse these in to just one
"TestPrefetchBufferThreshold" test.

It's a balancing act, but each of these tests requires starting up and shutting
down chrome, so we probably shouldn't make them quite this fine-grained, lest we
make the suite of ppapi tests too slow on the bots.

[Jay Civelli, 2012-01-17 17:31:12]

LGTM

http://codereview.chromium.org/9139076/diff/6001/ppapi/tests/test_url_loader.cc
File ppapi/tests/test_url_loader.cc (right):

http://codereview.chromium.org/9139076/diff/6001/ppapi/tests/test_url_loader....
ppapi/tests/test_url_loader.cc:882: int32_t rv = OpenUntrusted(request);
Nit: why not returning without the use of the local var?

[commit-bot: I haz the power, 2012-01-17 21:02:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/raymes@chromium.org/9139076/9001

[commit-bot: I haz the power, 2012-01-17 22:10:17]

Change committed as 117970