Change how the quota system computes the total poolsize for temporary storage

content/shell/browser/shell_content_browser_client.cc

Index: content/shell/browser/shell_content_browser_client.cc
diff --git a/content/shell/browser/shell_content_browser_client.cc b/content/shell/browser/shell_content_browser_client.cc
index d02f675ed58a0815ff59f5aa5a62c8e37619a1e5..d85671a00ee58103bfe75d34590f9e1865488cb3 100644
--- a/content/shell/browser/shell_content_browser_client.cc
+++ b/content/shell/browser/shell_content_browser_client.cc
@@ -253,6 +253,20 @@ ShellContentBrowserClient::CreateQuotaPermissionContext() {
   return new ShellQuotaPermissionContext();
 }
 
+void ShellContentBrowserClient::GetTemporaryStorageConfiguration(
+    BrowserContext* context,
+    const base::FilePath& partition_path, bool is_incognito,
+    const storage::TemporaryStorageConfigurationCallback& callback) {
+  // We use simple hard coded values for content_shell.
+  const int kMB = 1024 * 1024;
+  storage::TemporaryStorageConfiguration config;

[jsbell, 2016/10/17 23:56:54]

Lots of repetitions of these common values across test shells.

Maybe a TemporaryStorageConfiguration::InitForTesting() that pre-populates the
fields? 

[michaeln, 2016/10/26 21:47:53]

good idea, i haven't done this yet, i'll look at the commonalities and put
something somewhere

+  config.per_host_quota = 10 * kMB;
+  config.pool_size = 50 * kMB;
+  config.must_remain_available =  10 * kMB;
+  config.refresh_interval = base::TimeDelta::Max();
+  callback.Run(config);
+}
+
 void ShellContentBrowserClient::SelectClientCertificate(
     WebContents* web_contents,
     net::SSLCertRequestInfo* cert_request_info,