Fix bug which caused DNS time not to be reported by LoadTimingObserver.

chrome/browser/net/load_timing_observer_unittest.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/net/load_timing_observer.h"
 
 #include "base/compiler_specific.h"
 #include "base/format_macros.h"
 #include "base/string_util.h"
 #include "base/time.h"
@@ skipping 205 matching lines @@
 
   // Add resolver entry.
   AddStartConnectJobEntries(observer, 1);
   NetLog::Source connect_source(NetLog::SOURCE_CONNECT_JOB, 1);
   AddStartEntry(observer,
                 connect_source,
                 NetLog::TYPE_HOST_RESOLVER_IMPL,
                 NULL);
   current_time += TimeDelta::FromSeconds(2);
   AddEndEntry(observer, connect_source, NetLog::TYPE_HOST_RESOLVER_IMPL, NULL);
+  AddEndConnectJobEntries(observer, 1);
 
   // Bind to connect job.
   scoped_refptr<net::NetLogSourceParameter> params(
       new net::NetLogSourceParameter("connect_job", connect_source));
   AddStartEntry(observer,
                 source,
                 NetLog::TYPE_SOCKET_POOL_BOUND_TO_CONNECT_JOB,
                 params.get());
 
   LoadTimingObserver::URLRequestRecord* record =
@@ skipping 76 matching lines @@
   AddStartEntry(observer,
                 source,
                 NetLog::TYPE_SOCKET_POOL_BOUND_TO_SOCKET,
                 params.get());
 
   LoadTimingObserver::URLRequestRecord* record =
       observer.GetURLRequestRecord(0);
   ASSERT_EQ(1000, record->timing.ssl_start);
   ASSERT_EQ(3000, record->timing.ssl_end);
 }