base::Bind fixes

base::Bind fixes


BUG=none
TEST=none


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

[wtc, 2011-11-17 08:16:21]

LGTM.

http://codereview.chromium.org/8589012/diff/1/chrome/browser/net/chrome_url_r...
File chrome/browser/net/chrome_url_request_context.cc (right):

http://codereview.chromium.org/8589012/diff/1/chrome/browser/net/chrome_url_r...
chrome/browser/net/chrome_url_request_context.cc:168: &result));
Nit: align all the function arguments of base::Bind().

http://codereview.chromium.org/8589012/diff/1/chrome/browser/net/network_stats.h
File chrome/browser/net/network_stats.h (right):

http://codereview.chromium.org/8589012/diff/1/chrome/browser/net/network_stat...
chrome/browser/net/network_stats.h:197: // We use this factory to create timeout
tasks for socket's ReadData.
Is this comment still accurate?

http://codereview.chromium.org/8589012/diff/1/chrome/browser/net/predictor.cc
File chrome/browser/net/predictor.cc (right):

http://codereview.chromium.org/8589012/diff/1/chrome/browser/net/predictor.cc...
chrome/browser/net/predictor.cc:703: // ScopedRunnableMethodFactory instances
need to be created and destroyed
This comment should not mention ScopedRunnableMethodFactory any
more.

http://codereview.chromium.org/8589012/diff/1/chrome/browser/net/sdch_diction...
File chrome/browser/net/sdch_dictionary_fetcher.h (right):

http://codereview.chromium.org/8589012/diff/1/chrome/browser/net/sdch_diction...
chrome/browser/net/sdch_dictionary_fetcher.h:62: // bandwidth from the actual
page load.  Create delayed tasks to spread out
Is this comment still accurate?  Does weak_factory_ still
create delayed tasks?

[commit-bot: I haz the power, 2011-11-17 19:38:02]

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

[commit-bot: I haz the power, 2011-11-17 21:18:26]

Change committed as 110554