Stop doing unnecessary UTF-8 to UTF-16 conversions in JSONWriter.

chrome/browser/search/iframe_source.cc

Index: chrome/browser/search/iframe_source.cc
diff --git a/chrome/browser/search/iframe_source.cc b/chrome/browser/search/iframe_source.cc
index b44ed3046c88b9fdd8e88bca658548cb0b84201f..f00c1de17758488dfc54834d180acbbb4b7cb144 100644
--- a/chrome/browser/search/iframe_source.cc
+++ b/chrome/browser/search/iframe_source.cc
@@ -88,7 +88,7 @@ void IframeSource::SendJSWithOrigin(
   }
 
   std::string js_escaped_origin;
-  base::JsonDoubleQuote(origin, false, &js_escaped_origin);
+  base::EscapeJSONString(origin, false, &js_escaped_origin);

[Mark Mentovai, 2013/12/06 15:35:12]

Here is one place I will ask that question.

In fact, if you take my suggestion, you might get a compiler warning about
js_escaped_origin being unused. Good point, compiler! It IS unused. We don’t
need this EscapeJSONString at all.

[Robert Sesek, 2013/12/09 19:52:09]

Done.

   base::StringPiece template_js =
       ResourceBundle::GetSharedInstance().GetRawDataResource(resource_id);
   std::string response(template_js.as_string());