[MD settings] i18n source stream filtering

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 <stdint.h>
 
 #include <string>
@@ skipping 194 matching lines @@
 }
 
 void WebUIDataSourceImpl::DisableDenyXFrameOptions() {
   deny_xframe_options_ = false;
 }
 
 void WebUIDataSourceImpl::DisableI18nAndUseGzipForAllPaths() {
   use_gzip_for_all_paths_ = true;
 }
 
+const ui::TemplateReplacements* WebUIDataSourceImpl::GetReplacements() const {
+  return &replacements_;
+}
+
 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 38 matching lines @@
   int resource_id = default_resource_;
   std::map<std::string, int>::iterator result;
   // Remove the query string for named resource lookups.
   std::string file_path = path.substr(0, path.find_first_of('?'));
   result = path_to_idr_map_.find(file_path);
   if (result != path_to_idr_map_.end())
     resource_id = result->second;
   DCHECK_NE(resource_id, -1);
   scoped_refptr<base::RefCountedMemory> response(
       GetContentClient()->GetDataResourceBytes(resource_id));
-
-  // TODO(dschuyler): improve filtering of which resource to run template
-  // expansion upon. TODO(dbeam): make a streaming filter that works on gzipped
-  // content.
-  if (response.get() && GetMimeType(path) == "text/html" &&

[Dan Beam, 2016/12/01 23:32:23]

so how are we only running replacements on text/html without this block?

[dschuyler, 2016/12/02 20:14:00]

Done.

-      !source()->IsGzipped(path)) {
-    std::string replaced = ui::ReplaceTemplateExpressions(
-        base::StringPiece(response->front_as<char>(), response->size()),
-        replacements_);
-    response = base::RefCountedString::TakeString(&replaced);
-  }
-
   callback.Run(response.get());
 }
 
 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