Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2630)

Unified Diff: chrome/browser/net/dns_host_info_unittest.cc

Issue 2866026: Rename Dns prefetching files to Predictor files... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/net/dns_host_info.cc ('k') | chrome/browser/net/dns_master.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/dns_host_info_unittest.cc
===================================================================
--- chrome/browser/net/dns_host_info_unittest.cc (revision 50915)
+++ chrome/browser/net/dns_host_info_unittest.cc (working copy)
@@ -1,129 +0,0 @@
-// Copyright (c) 2006-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.
-
-// Single threaded tests of UrlInfo functionality.
-
-#include <time.h>
-#include <string>
-
-#include "base/platform_thread.h"
-#include "chrome/browser/net/dns_host_info.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-using base::TimeDelta;
-
-namespace {
-
-class DnsHostInfoTest : public testing::Test {
-};
-
-typedef chrome_browser_net::UrlInfo UrlInfo;
-
-TEST(DnsHostInfoTest, StateChangeTest) {
- UrlInfo info_practice, info;
- GURL url1("http://domain1.com:80"), url2("https://domain2.com:443");
-
- // First load DLL, so that their load time won't interfere with tests.
- // Some tests involve timing function performance, and DLL time can overwhelm
- // test durations (which are considering network vs cache response times).
- info_practice.SetUrl(url2);
- info_practice.SetQueuedState(UrlInfo::UNIT_TEST_MOTIVATED);
- info_practice.SetAssignedState();
- info_practice.SetFoundState();
- PlatformThread::Sleep(500); // Allow time for DLLs to fully load.
-
- // Complete the construction of real test object.
- info.SetUrl(url1);
-
- EXPECT_TRUE(info.NeedsDnsUpdate()) << "error in construction state";
- info.SetQueuedState(UrlInfo::UNIT_TEST_MOTIVATED);
- EXPECT_FALSE(info.NeedsDnsUpdate())
- << "update needed after being queued";
- info.SetAssignedState();
- EXPECT_FALSE(info.NeedsDnsUpdate());
- info.SetFoundState();
- EXPECT_FALSE(info.NeedsDnsUpdate())
- << "default expiration time is TOOOOO short";
-
- // Note that time from ASSIGNED to FOUND was VERY short (probably 0ms), so the
- // object should conclude that no network activity was needed. As a result,
- // the required time till expiration will be halved (guessing that we were
- // half way through having the cache expire when we did the lookup.
- EXPECT_LT(info.resolve_duration().InMilliseconds(),
- UrlInfo::kMaxNonNetworkDnsLookupDuration.InMilliseconds())
- << "Non-net time is set too low";
-
- info.set_cache_expiration(TimeDelta::FromMilliseconds(300));
- EXPECT_FALSE(info.NeedsDnsUpdate()) << "expiration time not honored";
- PlatformThread::Sleep(80); // Not enough time to pass our 300ms mark.
- EXPECT_FALSE(info.NeedsDnsUpdate()) << "expiration time not honored";
-
- // That was a nice life when the object was found.... but next time it won't
- // be found. We'll sleep for a while, and then come back with not-found.
- info.SetQueuedState(UrlInfo::UNIT_TEST_MOTIVATED);
- info.SetAssignedState();
- EXPECT_FALSE(info.NeedsDnsUpdate());
- // Greater than minimal expected network latency on DNS lookup.
- PlatformThread::Sleep(25);
- info.SetNoSuchNameState();
- EXPECT_FALSE(info.NeedsDnsUpdate())
- << "default expiration time is TOOOOO short";
-
- // Note that now we'll actually utilize an expiration of 300ms,
- // since there was detected network activity time during lookup.
- // We're assuming the caching just started with our lookup.
- PlatformThread::Sleep(80); // Not enough time to pass our 300ms mark.
- EXPECT_FALSE(info.NeedsDnsUpdate()) << "expiration time not honored";
- // Still not past our 300ms mark (only about 4+2ms)
- PlatformThread::Sleep(80);
- EXPECT_FALSE(info.NeedsDnsUpdate()) << "expiration time not honored";
- PlatformThread::Sleep(150);
- EXPECT_TRUE(info.NeedsDnsUpdate()) << "expiration time not honored";
-}
-
-// When a system gets "congested" relative to DNS, it means it is doing too many
-// DNS resolutions, and bogging down the system. When we detect such a
-// situation, we divert the sequence of states a UrlInfo instance moves
-// through. Rather than proceeding from QUEUED (waiting in a name queue for a
-// worker thread that can resolve the name) to ASSIGNED (where a worker thread
-// actively resolves the name), we enter the ASSIGNED state (without actually
-// getting sent to a resolver thread) and reset our state to what it was before
-// the corresponding name was put in the work_queue_. This test drives through
-// the state transitions used in such congestion handling.
-TEST(DnsHostInfoTest, CongestionResetStateTest) {
- UrlInfo info;
- GURL url("http://domain1.com:80");
-
- info.SetUrl(url);
- info.SetQueuedState(UrlInfo::UNIT_TEST_MOTIVATED);
- info.SetAssignedState();
- EXPECT_TRUE(info.is_assigned());
-
- info.RemoveFromQueue(); // Do the reset.
- EXPECT_FALSE(info.is_assigned());
-
- // Since this was a new info instance, and it never got resolved, we land back
- // in a PENDING state rather than FOUND or NO_SUCH_NAME.
- EXPECT_FALSE(info.was_found());
- EXPECT_FALSE(info.was_nonexistant());
-
- // Make sure we're completely re-usable, by going throug a normal flow.
- info.SetQueuedState(UrlInfo::UNIT_TEST_MOTIVATED);
- info.SetAssignedState();
- info.SetFoundState();
- EXPECT_TRUE(info.was_found());
-
- // Use the congestion flow, and check that we end up in the found state.
- info.SetQueuedState(UrlInfo::UNIT_TEST_MOTIVATED);
- info.SetAssignedState();
- info.RemoveFromQueue(); // Do the reset.
- EXPECT_FALSE(info.is_assigned());
- EXPECT_TRUE(info.was_found()); // Back to what it was before being queued.
-}
-
-
-// TODO(jar): Add death test for illegal state changes, and also for setting
-// hostname when already set.
-
-} // namespace
« no previous file with comments | « chrome/browser/net/dns_host_info.cc ('k') | chrome/browser/net/dns_master.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698