Convert uses of int ms to TimeDelta in chrome/browser and net.

Convert uses of int ms to TimeDelta in chrome/browser and net.


R=jar@chromium.org
BUG=108171
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=124890

[jar (doing other things), 2012-03-03 22:54:43]

LGTM

Also noted two places you might want to go further (maybe in this or another
CL).

https://chromiumcodereview.appspot.com/9572036/diff/1/chrome/browser/metrics/...
File chrome/browser/metrics/histogram_synchronizer.cc (right):

https://chromiumcodereview.appspot.com/9572036/diff/1/chrome/browser/metrics/...
chrome/browser/metrics/histogram_synchronizer.cc:81: int wait_time) {
I *think* you should propagate the chage further back here... and avoid the
translation in the post.

WDYT?

https://chromiumcodereview.appspot.com/9572036/diff/1/net/curvecp/client_pack...
File net/curvecp/client_packetizer.cc (right):

https://chromiumcodereview.appspot.com/9572036/diff/1/net/curvecp/client_pack...
net/curvecp/client_packetizer.cc:316: void ClientPacketizer::StartHelloTimer(int
milliseconds) {
Another *possible* candidate for using TimeDelta

[Ted Vessenes, 2012-03-04 05:48:37]

I think those changes are largely justified, so I updated as such.

[commit-bot: I haz the power, 2012-03-04 05:48:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tedvessenes@gmail.com/9572036/5001

[commit-bot: I haz the power, 2012-03-04 07:18:35]

Change committed as 124890

[Ryan Sleevi, 2012-03-04 07:37:13]

Had to revert this. Looks like those last minute changes were missing some
proper includes.

In file included from net/curvecp/client_packetizer.cc:5:
./net/curvecp/client_packetizer.h:69:error: 'base::TimeDelta' has not been
declared
net/curvecp/client_packetizer.cc: In member function 'int
net::ClientPacketizer::DoWaitingCookie()':
net/curvecp/client_packetizer.cc:239:error: no matching function for call to
'net::ClientPacketizer::StartHelloTimer(base::TimeDelta)'
./net/curvecp/client_packetizer.h:69: note: candidates are: void
net::ClientPacketizer::StartHelloTimer(int)
net/curvecp/client_packetizer.cc: At global scope:
net/curvecp/client_packetizer.cc:317:error: prototype for 'void
net::ClientPacketizer::StartHelloTimer(base::TimeDelta)' does not match any in
class 'net::ClientPacketizer'
./net/curvecp/client_packetizer.h:69:error: candidate is: void
net::ClientPacketizer::StartHelloTimer(int)
make: *** [out/Release/obj.target/curvecp/net/curvecp/client_packetizer.o] Error
1