Bind weak ptr in PostStartDnsProbeTask, not StartDnsProbe

chrome/browser/net/net_error_tab_helper.cc

Index: chrome/browser/net/net_error_tab_helper.cc
diff --git a/chrome/browser/net/net_error_tab_helper.cc b/chrome/browser/net/net_error_tab_helper.cc
index db6e3119b96c96ff4ea6cf280fecee7f2ba6edac..529c0b1d644aa9660949cd166aa2b4b171a2ff79 100644
--- a/chrome/browser/net/net_error_tab_helper.cc
+++ b/chrome/browser/net/net_error_tab_helper.cc
@@ -51,28 +51,25 @@ bool GetEnabledByTrial() {
 }
 
 void DnsProbeCallback(

[Ryan Sleevi, 2013/02/12 00:29:31]

nit: It would be helpful to rename this function to something more meaningful
("callback" usually refers to bound functions, not function names themselves)

"OnDnsProbeFinishedOnIOThread" is one more descriptive name for this.

[Deprecated (see juliatuttle), 2013/02/13 03:17:57]

Done.

-    base::WeakPtr<NetErrorTabHelper> tab_helper,
-    base::Callback<void(NetErrorTabHelper*, DnsProbeResult)> callback,
+    base::Callback<void(DnsProbeResult)> callback,

[Ryan Sleevi, 2013/02/12 00:29:31]

nit: const base::Callback<void(DnsProbeResult)>& callback,

Also on line 65

[Deprecated (see juliatuttle), 2013/02/13 03:17:57]

Done.

     DnsProbeResult result) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   BrowserThread::PostTask(
       BrowserThread::UI,
       FROM_HERE,
-      base::Bind(callback, tab_helper, result));
+      base::Bind(callback, result));
 }
 
 void StartDnsProbe(

[Ryan Sleevi, 2013/02/12 00:29:31]

nit: StartDnsProbeOnIOThread ?

[Deprecated (see juliatuttle), 2013/02/13 03:17:57]

Done.

-    base::WeakPtr<NetErrorTabHelper> tab_helper,
-    base::Callback<void(NetErrorTabHelper*, DnsProbeResult)> callback,
+    base::Callback<void(DnsProbeResult)> callback,
     IOThread* io_thread) {

[Ryan Sleevi, 2013/02/12 00:29:31]

Since you're using g_browser_process->io_thread() on line 194, why bother
binding IOThread here for the function?

Why not just grab g_browser_process->io_thread() here (or more aptly, line 68),
while you're executing on the IO thread?

[Deprecated (see juliatuttle), 2013/02/13 03:17:57]

Done.

   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   DnsProbeService* probe_service =
       io_thread->globals()->dns_probe_service.get();
-  probe_service->ProbeDns(base::Bind(&DnsProbeCallback,
-                                     tab_helper,
-                                     callback));
+
+  probe_service->ProbeDns(base::Bind(&DnsProbeCallback, callback));
 }
 
 }  // namespace
@@ -189,10 +186,12 @@ void NetErrorTabHelper::PostStartDnsProbeTask() {
   BrowserThread::PostTask(
       BrowserThread::IO,
       FROM_HERE,
-      base::Bind(&StartDnsProbe,
-                 weak_factory_.GetWeakPtr(),
-                 base::Bind(&NetErrorTabHelper::OnDnsProbeFinished),
-                 g_browser_process->io_thread()));
+      base::Bind(
+          &StartDnsProbe,
+          base::Bind(
+              &NetErrorTabHelper::OnDnsProbeFinished,
+              weak_factory_.GetWeakPtr()),
+          g_browser_process->io_thread()));
 }
 
 void NetErrorTabHelper::OnDnsProbeFinished(DnsProbeResult result) {