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

Side by Side Diff: net/http/http_server_properties_manager_unittest.cc

Issue 2699003002: Clear cached QUIC network stats when a QUIC handshake fails. (Closed)
Patch Set: Created 3 years, 10 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/http/http_server_properties_manager.h" 5 #include "net/http/http_server_properties_manager.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/json/json_reader.h" 10 #include "base/json/json_reader.h"
(...skipping 889 matching lines...) Expand 10 before | Expand all | Expand 10 after
900 EXPECT_TRUE(pref_test_task_runner_->HasPendingTask()); 900 EXPECT_TRUE(pref_test_task_runner_->HasPendingTask());
901 pref_test_task_runner_->FastForwardUntilNoTasksRemain(); 901 pref_test_task_runner_->FastForwardUntilNoTasksRemain();
902 EXPECT_FALSE(net_test_task_runner_->HasPendingTask()); 902 EXPECT_FALSE(net_test_task_runner_->HasPendingTask());
903 EXPECT_FALSE(pref_test_task_runner_->HasPendingTask()); 903 EXPECT_FALSE(pref_test_task_runner_->HasPendingTask());
904 904
905 Mock::VerifyAndClearExpectations(http_server_props_manager_.get()); 905 Mock::VerifyAndClearExpectations(http_server_props_manager_.get());
906 906
907 const ServerNetworkStats* stats2 = 907 const ServerNetworkStats* stats2 =
908 http_server_props_manager_->GetServerNetworkStats(mail_server); 908 http_server_props_manager_->GetServerNetworkStats(mail_server);
909 EXPECT_EQ(10, stats2->srtt.ToInternalValue()); 909 EXPECT_EQ(10, stats2->srtt.ToInternalValue());
910
911 ExpectPrefsUpdate(1);
912 ExpectScheduleUpdatePrefsOnNetworkThread();
913
914 http_server_props_manager_->ClearServerNetworkStats(mail_server);
915
916 // Run the task.
917 EXPECT_FALSE(pref_test_task_runner_->HasPendingTask());
918 EXPECT_TRUE(net_test_task_runner_->HasPendingTask());
919 net_test_task_runner_->FastForwardUntilNoTasksRemain();
920 EXPECT_TRUE(pref_test_task_runner_->HasPendingTask());
921 pref_test_task_runner_->FastForwardUntilNoTasksRemain();
922 EXPECT_FALSE(net_test_task_runner_->HasPendingTask());
923 EXPECT_FALSE(pref_test_task_runner_->HasPendingTask());
924
925 Mock::VerifyAndClearExpectations(http_server_props_manager_.get());
926 EXPECT_EQ(nullptr,
927 http_server_props_manager_->GetServerNetworkStats(mail_server));
910 } 928 }
911 929
912 TEST_P(HttpServerPropertiesManagerTest, QuicServerInfo) { 930 TEST_P(HttpServerPropertiesManagerTest, QuicServerInfo) {
913 ExpectPrefsUpdate(1); 931 ExpectPrefsUpdate(1);
914 ExpectScheduleUpdatePrefsOnNetworkThread(); 932 ExpectScheduleUpdatePrefsOnNetworkThread();
915 933
916 QuicServerId mail_quic_server_id("mail.google.com", 80); 934 QuicServerId mail_quic_server_id("mail.google.com", 80);
917 EXPECT_EQ(nullptr, 935 EXPECT_EQ(nullptr,
918 http_server_props_manager_->GetQuicServerInfo(mail_quic_server_id)); 936 http_server_props_manager_->GetQuicServerInfo(mail_quic_server_id));
919 std::string quic_server_info1("quic_server_info1"); 937 std::string quic_server_info1("quic_server_info1");
(...skipping 582 matching lines...) Expand 10 before | Expand all | Expand 10 after
1502 pref_test_task_runner_->FastForwardUntilNoTasksRemain(); 1520 pref_test_task_runner_->FastForwardUntilNoTasksRemain();
1503 EXPECT_FALSE(net_test_task_runner_->HasPendingTask()); 1521 EXPECT_FALSE(net_test_task_runner_->HasPendingTask());
1504 EXPECT_FALSE(pref_test_task_runner_->HasPendingTask()); 1522 EXPECT_FALSE(pref_test_task_runner_->HasPendingTask());
1505 Mock::VerifyAndClearExpectations(http_server_props_manager_.get()); 1523 Mock::VerifyAndClearExpectations(http_server_props_manager_.get());
1506 http_server_props_manager_.reset(); 1524 http_server_props_manager_.reset();
1507 EXPECT_FALSE(net_test_task_runner_->HasPendingTask()); 1525 EXPECT_FALSE(net_test_task_runner_->HasPendingTask());
1508 EXPECT_FALSE(pref_test_task_runner_->HasPendingTask()); 1526 EXPECT_FALSE(pref_test_task_runner_->HasPendingTask());
1509 } 1527 }
1510 1528
1511 } // namespace net 1529 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698