base::Bind() conversion for crash handler/uploader.

base::Bind() conversion for crash handler/uploader.

BUG=none
TEST=trybots


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

[stuartmorgan, 2011-11-21 08:36:16]

Upload list changes LGTM with nit; I know absolutely nothing about the Linux
crash handler, so I'm not the right person to review that part of the change.

http://codereview.chromium.org/8536055/diff/1/chrome/browser/crash_upload_lis...
File chrome/browser/crash_upload_list.cc (right):

http://codereview.chromium.org/8536055/diff/1/chrome/browser/crash_upload_lis...
chrome/browser/crash_upload_list.cc:57:
base::Bind(&CrashUploadList::InformDelegateOfCompletion, this));
If you are going to change the style above, you should do it here too so you
aren't making the file less consistent.

[dcheng, 2011-11-21 17:15:50]

http://codereview.chromium.org/8536055/diff/1/chrome/browser/crash_upload_lis...
File chrome/browser/crash_upload_list.cc (right):

http://codereview.chromium.org/8536055/diff/1/chrome/browser/crash_upload_lis...
chrome/browser/crash_upload_list.cc:57:
base::Bind(&CrashUploadList::InformDelegateOfCompletion, this));
On 2011/11/21 08:36:16, stuartmorgan wrote:
> If you are going to change the style above, you should do it here too so you
> aren't making the file less consistent.

Done.

[Lei Zhang, 2011-11-22 19:53:54]

LGTM

[commit-bot: I haz the power, 2011-11-22 19:59:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dcheng@chromium.org/8536055/4001

[commit-bot: I haz the power, 2011-11-22 21:44:24]

Change committed as 111223