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

Unified Diff: net/sdch/sdch_owner_unittest.cc

Issue 962313003: [net]: Disable SdchOwner's MemoryPressureListener to investigate http://crbug.com/447208. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 side-by-side diff with in-line comments
Download patch
« net/sdch/sdch_owner.cc ('K') | « net/sdch/sdch_owner.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/sdch/sdch_owner_unittest.cc
diff --git a/net/sdch/sdch_owner_unittest.cc b/net/sdch/sdch_owner_unittest.cc
index e89533a361c4a45477c6affcf16a87ea2820031b..66a1a4d01248b4d9792e1a0810d2424ada1c275f 100644
--- a/net/sdch/sdch_owner_unittest.cc
+++ b/net/sdch/sdch_owner_unittest.cc
@@ -460,6 +460,8 @@ TEST_F(SdchOwnerTest, ClearReturnsSpace) {
EXPECT_TRUE(CreateAndAddDictionary(kMaxSizeForTesting, nullptr));
}
+#if 0
+// Remove memory_pressure_listener_ to investigate http://crbug.com/447208
// Confirm memory pressure gets all the space back.
TEST_F(SdchOwnerTest, MemoryPressureReturnsSpace) {
std::string server_hash_d1;
@@ -487,5 +489,6 @@ TEST_F(SdchOwnerTest, MemoryPressureReturnsSpace) {
// Addition should now succeed.
EXPECT_TRUE(CreateAndAddDictionary(kMaxSizeForTesting, nullptr));
}
+#endif
} // namespace net
« net/sdch/sdch_owner.cc ('K') | « net/sdch/sdch_owner.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698