diff options
author | ericroman@google.com <ericroman@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-14 03:01:09 +0000 |
---|---|---|
committer | ericroman@google.com <ericroman@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-14 03:01:09 +0000 |
commit | 54e1377ea309c5d8ab318ee9691b46249d7441b6 (patch) | |
tree | 8471e44056027dff255ff5b22ba083a9482b41c3 /net/base/load_log_util_unittest.cc | |
parent | 672eca0fd191acd39c28430d1ed1f327594faff8 (diff) | |
download | chromium_src-54e1377ea309c5d8ab318ee9691b46249d7441b6.zip chromium_src-54e1377ea309c5d8ab318ee9691b46249d7441b6.tar.gz chromium_src-54e1377ea309c5d8ab318ee9691b46249d7441b6.tar.bz2 |
Implement LoadLog, and hook up HostResolverImpl to LoadLog.
The functionality in load_log_util.h is currently unused, but since it motivates the implementation of LoadLog, figured it should be included with this review.
Note that I decided against defining the loggable event types as LoadStates, since I wanted more flexibility in adding crazy stuff, and they really seem too implementation specific to be in LoadStates.
BUG=http://crbug.com/14478
TEST=net_unittests
Review URL: http://codereview.chromium.org/165404
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23412 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/base/load_log_util_unittest.cc')
-rw-r--r-- | net/base/load_log_util_unittest.cc | 57 |
1 files changed, 57 insertions, 0 deletions
diff --git a/net/base/load_log_util_unittest.cc b/net/base/load_log_util_unittest.cc new file mode 100644 index 0000000..fbaa77b --- /dev/null +++ b/net/base/load_log_util_unittest.cc @@ -0,0 +1,57 @@ +// Copyright (c) 2009 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 "net/base/load_log_unittest.h" +#include "net/base/load_log_util.h" +#include "testing/gtest/include/gtest/gtest.h" + +namespace net { +namespace { + +TEST(LoadLogUtilTest, Basic) { + scoped_refptr<LoadLog> log(new LoadLog); + + log->Add(MakeTime(1), LoadLog::TYPE_HOST_RESOLVER_IMPL, LoadLog::PHASE_BEGIN); + log->Add(MakeTime(5), LoadLog::TYPE_HOST_RESOLVER_IMPL_OBSERVER_ONSTART, + LoadLog::PHASE_BEGIN); + log->Add(MakeTime(8), LoadLog::TYPE_HOST_RESOLVER_IMPL_OBSERVER_ONSTART, + LoadLog::PHASE_END); + log->Add(MakeTime(12), LoadLog::TYPE_CANCELLED, LoadLog::PHASE_NONE); + log->Add(MakeTime(131), LoadLog::TYPE_HOST_RESOLVER_IMPL, LoadLog::PHASE_END); + + EXPECT_EQ( + "t= 1: +HOST_RESOLVER_IMPL [dt=130]\n" + "t= 5: +HOST_RESOLVER_IMPL_OBSERVER_ONSTART [dt= 3]\n" + "t= 8: -HOST_RESOLVER_IMPL_OBSERVER_ONSTART\n" + "t= 12: CANCELLED\n" + "t=131: -HOST_RESOLVER_IMPL", + LoadLogUtil::PrettyPrintAsEventTree(log)); +} + +TEST(LoadLogUtilTest, UnmatchedOpen) { + scoped_refptr<LoadLog> log(new LoadLog); + + log->Add(MakeTime(3), LoadLog::TYPE_HOST_RESOLVER_IMPL, LoadLog::PHASE_BEGIN); + // Note that there is no matching call to PHASE_END for all of the following. + log->Add(MakeTime(6), LoadLog::TYPE_HOST_RESOLVER_IMPL_OBSERVER_ONSTART, + LoadLog::PHASE_BEGIN); + log->Add(MakeTime(7), LoadLog::TYPE_HOST_RESOLVER_IMPL_OBSERVER_ONSTART, + LoadLog::PHASE_BEGIN); + log->Add(MakeTime(8), LoadLog::TYPE_HOST_RESOLVER_IMPL_OBSERVER_ONSTART, + LoadLog::PHASE_BEGIN); + log->Add(MakeTime(10), LoadLog::TYPE_CANCELLED, LoadLog::PHASE_NONE); + log->Add(MakeTime(16), LoadLog::TYPE_HOST_RESOLVER_IMPL, LoadLog::PHASE_END); + + EXPECT_EQ( + "t= 3: +HOST_RESOLVER_IMPL [dt=13]\n" + "t= 6: +HOST_RESOLVER_IMPL_OBSERVER_ONSTART [dt=10]\n" + "t= 7: +HOST_RESOLVER_IMPL_OBSERVER_ONSTART [dt= 9]\n" + "t= 8: +HOST_RESOLVER_IMPL_OBSERVER_ONSTART [dt= 8]\n" + "t=10: CANCELLED\n" + "t=16: -HOST_RESOLVER_IMPL", + LoadLogUtil::PrettyPrintAsEventTree(log)); +} + +} // namespace +} // namespace net |