NetLogEventParameter to Callback refactoring 1,

net/base/multi_threaded_cert_verifier.cc

Index: net/base/multi_threaded_cert_verifier.cc
===================================================================
--- net/base/multi_threaded_cert_verifier.cc	(revision 141407)
+++ net/base/multi_threaded_cert_verifier.cc	(working copy)
@@ -152,7 +152,7 @@
 
   // Returns the certificate being verified. May only be called /before/
   // Start() is called.
-  X509Certificate* certificate() const { return cert_; }
+  scoped_refptr<X509Certificate> certificate() const { return cert_; }

[eroman, 2012/06/11 23:42:25]

Is this change necessary? Seems fine and I'm not suggesting you change it, but
wondering if it was actually necessary for correctness.

My understanding is it wouldn't matter if the extra reference was held from the
perspective of NetLog callback.

[mmenke, 2012/06/12 00:42:19]

It's not...But if it someone was being bad and keeping around a pointer to a ref
counted object before adding the first reference to it, and I stored it in a
refcounted temporary, bad things would happen.  Making this explicit makes life
less scary with a smart IDE that shows the return type of functions on mouse
over.

That having been said, since I've removed the code to add a ref, per your
suggestion, I've reverted this, since it's no longer an issue.

 
   bool Start() {
     DCHECK_EQ(MessageLoop::current(), origin_loop_);
@@ -263,9 +263,9 @@
       : start_time_(base::TimeTicks::Now()),
         worker_(worker),
         net_log_(net_log) {
-    scoped_refptr<NetLog::EventParameters> params(
-        new X509CertificateNetLogParam(worker_->certificate()));
-    net_log_.BeginEvent(NetLog::TYPE_CERT_VERIFIER_JOB, params);
+    net_log_.BeginEvent(NetLog::TYPE_CERT_VERIFIER_JOB,
+        base::Bind(&NetLogX509CertificateNetLogCallback,
+                   worker_->certificate()));
   }
 
   ~CertVerifierJob() {
@@ -280,8 +280,7 @@
   void AddRequest(CertVerifierRequest* request) {
     request->net_log().AddEvent(
         NetLog::TYPE_CERT_VERIFIER_REQUEST_BOUND_TO_JOB,
-        make_scoped_refptr(new NetLogSourceParameter(
-          "source_dependency", net_log_.source())));
+        net_log_.source().ToEventParametersCallback());
 
     requests_.push_back(request);
   }