Add timeout for safebrowsing test.

Add timeout for safebrowsing test.


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

[lzheng, 2010-10-13 18:51:09]

I started a local buildbot using this change, although I can force local build,
but I could not send a change for it to build. 
http://lzheng.mtv.corp.google.com:9010/waterfall

The command I used to send to the bot is:
gcl try test_tmp --host lzheng.mtv.corp.google.com --port 9012 

Any idea what could be wrong?

[M-A Ruel, 2010-10-13 18:56:09]

I thought you wanted to do that only on Windows, you can look for
self._target_platform == 'win32' if you want.

In any case lgtm.

[lzheng, 2010-10-14 16:31:49]

I believe it would be easier to keep track by using just one value across all
platforms. I will keep it this way since you are okay with either way.

Also, I changed 40 to 40000, since the time is msec, not sec.




On 2010/10/13 18:56:09, Marc-Antoine Ruel wrote:
> I thought you wanted to do that only on Windows, you can look for
> self._target_platform == 'win32' if you want.
> 
> In any case lgtm.

[M-A Ruel, 2010-10-14 16:39:04]

lgtm