Change the way that gzipped resources are loaded from resources.pak

content/browser/webui/web_ui_data_source_impl.cc

 // Copyright (c) 2012 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.
 
 #include "content/browser/webui/web_ui_data_source_impl.h"
 
 #include <stddef.h>
 
 #include <string>
 
@@ skipping 56 matching lines @@
     return URLDataSource::GetContentSecurityPolicyObjectSrc();
   }
   std::string GetContentSecurityPolicyChildSrc() const override {
     if (parent_->frame_src_set_)
       return parent_->frame_src_;
     return URLDataSource::GetContentSecurityPolicyChildSrc();
   }
   bool ShouldDenyXFrameOptions() const override {
     return parent_->deny_xframe_options_;
   }
+  bool IsGzipped(const std::string& path) const override {
+    return parent_->use_gzip_for_all_paths_ && path != parent_->json_path_;

[smaier, 2016/07/15 14:44:47]

Why do we care if we match the json_path_ of the parent?

[Dan Beam, 2016/07/15 16:44:40]

because things like string.js aren't gzipped

+  }
 
  private:
   WebUIDataSourceImpl* parent_;
 };
 
 WebUIDataSourceImpl::WebUIDataSourceImpl(const std::string& source_name)
     : URLDataSourceImpl(source_name, new InternalDataSource(this)),
       source_name_(source_name),
       default_resource_(-1),
       add_csp_(true),
       object_src_set_(false),
       frame_src_set_(false),
       deny_xframe_options_(true),
       add_load_time_data_defaults_(true),
-      replace_existing_source_(true) {}
+      replace_existing_source_(true),
+      use_gzip_for_all_paths_(false) {}
 
 WebUIDataSourceImpl::~WebUIDataSourceImpl() {
 }
 
 void WebUIDataSourceImpl::AddString(const std::string& name,
                                     const base::string16& value) {
   // TODO(dschuyler): Share only one copy of these strings.
   localized_strings_.SetString(name, value);
   replacements_[name] = base::UTF16ToUTF8(value);
 }
@@ skipping 70 matching lines @@
 void WebUIDataSourceImpl::OverrideContentSecurityPolicyChildSrc(
     const std::string& data) {
   frame_src_set_ = true;
   frame_src_ = data;
 }
 
 void WebUIDataSourceImpl::DisableDenyXFrameOptions() {
   deny_xframe_options_ = false;
 }
 
+void WebUIDataSourceImpl::UseGzipForAllPaths() {
+  use_gzip_for_all_paths_ = true;
+}
+
 std::string WebUIDataSourceImpl::GetSource() const {
   return source_name_;
 }
 
 std::string WebUIDataSourceImpl::GetMimeType(const std::string& path) const {
   // Remove the query string for to determine the mime type.
   std::string file_path = path.substr(0, path.find_first_of('?'));
 
   if (base::EndsWith(file_path, ".css", base::CompareCase::INSENSITIVE_ASCII))
     return "text/css";
@@ skipping 60 matching lines @@
 }
 
 void WebUIDataSourceImpl::SendLocalizedStringsAsJSON(
     const URLDataSource::GotDataCallback& callback) {
   std::string template_data;
   webui::AppendJsonJS(&localized_strings_, &template_data);
   callback.Run(base::RefCountedString::TakeString(&template_data));
 }
 
 }  // namespace content