MD Downloads: gzip vulcanized resources via compress="gzip"

content/public/browser/web_ui_data_source.h

 // Copyright (c) 2013 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_WEB_UI_DATA_SOURCE_H_
 #define CONTENT_PUBLIC_BROWSER_WEB_UI_DATA_SOURCE_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 87 matching lines @@
   virtual void DisableReplaceExistingSource() = 0;
   virtual void DisableContentSecurityPolicy() = 0;
   virtual void OverrideContentSecurityPolicyScriptSrc(
       const std::string& data) = 0;
   virtual void OverrideContentSecurityPolicyObjectSrc(
       const std::string& data) = 0;
   virtual void OverrideContentSecurityPolicyChildSrc(
       const std::string& data) = 0;
   virtual void DisableDenyXFrameOptions() = 0;
 
-  // Tells the loading code that resources are gzipped on disk. TODO(dbeam):
-  // write a streaming $i18n{} replacer and remove the "DisableI18n" part.
-  virtual void DisableI18nAndUseGzipForAllPaths() = 0;
+  // Tells the loading code that resources are gzipped on disk.
+  virtual void UseGzipForAllPaths() = 0;
+
+  // When UseGzipForAllPaths is enabled, exclude the given |path|.
+  virtual void ExcludePathFromGzip(const std::string& path) = 0;

[jam, 2016/12/21 21:18:31]

do we really need two methods? can't UseGzipForAllPaths just take a vector for
exceptions? then there'd be no need to know what the right order for calling
these methods are etc

[Dan Beam, 2016/12/21 22:22:05]

how does

virtual void UseGzip(const std::vector<std::string>& excluded_paths) = 0;

sound to you?

[Dan Beam, 2016/12/21 23:06:34]

Done. (*except that I forgot excluded_paths_ was implemented as an
unordered_set, which seems fine for this)

 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_WEB_UI_DATA_SOURCE_H_