First pass at mobile (and other) SDCH limit revisions.

net/base/sdch_manager.cc

Index: net/base/sdch_manager.cc
diff --git a/net/base/sdch_manager.cc b/net/base/sdch_manager.cc
index ad260a58e3f0c28d099c33ec65e677f6a0f85818..a3545835de7c579e1daf585f829e8afccf256909 100644
--- a/net/base/sdch_manager.cc
+++ b/net/base/sdch_manager.cc
@@ -17,10 +17,17 @@ namespace net {
 
 //------------------------------------------------------------------------------
 // static
-const size_t SdchManager::kMaxDictionarySize = 1000000;
 
+// Adjust SDCH limits downwards for mobile.
+#if defined(OS_ANDROID) || defined(OS_IOS)
 // static
-const size_t SdchManager::kMaxDictionaryCount = 20;
+const size_t SdchManager::kMaxDictionaryCount = 2;

[cbentzel, 2014/06/18 20:47:43]

Limit we agreed on was 150KB max for Android - this looks like 600KB. 

[Randy Smith (Not in Mondays), 2014/06/18 21:16:40]

Fixed.

+const size_t SdchManager::kMaxDictionarySize = 300 * 1000;
+#else
+// static
+const size_t SdchManager::kMaxDictionaryCount = 5;

[cbentzel, 2014/06/18 20:47:43]

Why did these change?

[Randy Smith (Not in Mondays), 2014/06/18 21:16:40]

Because I think desktop chrome is a bit of a pig around SDCH at the moment.  But
I probably shouldn't change things unless I have some data to drive that with;
reverted.

+const size_t SdchManager::kMaxDictionarySize = 500 * 1000;
+#endif
 
 // static
 bool SdchManager::g_sdch_enabled_ = true;