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

chrome/browser/extensions/activity_log/activity_log_unittest.cc

Index: chrome/browser/extensions/activity_log/activity_log_unittest.cc
diff --git a/chrome/browser/extensions/activity_log/activity_log_unittest.cc b/chrome/browser/extensions/activity_log/activity_log_unittest.cc
index 2d11ca53830a6d53ceb6b2e0fe75d510f3078245..fedd5feee9c8430982bf89a712d1919ffdf9ca89 100644
--- a/chrome/browser/extensions/activity_log/activity_log_unittest.cc
+++ b/chrome/browser/extensions/activity_log/activity_log_unittest.cc
@@ -117,7 +117,7 @@ class ActivityLogTest : public ChromeRenderViewHostTestHarness {
     ASSERT_EQ(4U, i->size());
     scoped_refptr<Action> action = i->at(0);
     ASSERT_EQ("XMLHttpRequest.open", action->api_name());
-    ASSERT_EQ("[\"POST\",\"\\u003Carg_url\\u003E\"]",
+    ASSERT_EQ("[\"POST\",\"\\u003Carg_url>\"]",

[Nico, 2013/12/09 21:20:50]

Huh, why does < stay escaped but > doesn't?

[Robert Sesek, 2013/12/09 21:21:45]

See earlier review thread. To prevent script execution, only < needs to be
escaped, and we can save a few bytes by not escaping >.

               ActivityLogPolicy::Util::Serialize(action->args()));
     ASSERT_EQ("http://api.google.com/", action->arg_url().spec());
     // Test that the dom_verb field was changed to XHR (from METHOD).  This
@@ -131,7 +131,7 @@ class ActivityLogTest : public ChromeRenderViewHostTestHarness {
 
     action = i->at(1);
     ASSERT_EQ("XMLHttpRequest.open", action->api_name());
-    ASSERT_EQ("[\"POST\",\"\\u003Carg_url\\u003E\"]",
+    ASSERT_EQ("[\"POST\",\"\\u003Carg_url>\"]",
               ActivityLogPolicy::Util::Serialize(action->args()));
     ASSERT_EQ("http://www.google.com/api/", action->arg_url().spec());
 
@@ -143,7 +143,7 @@ class ActivityLogTest : public ChromeRenderViewHostTestHarness {
 
     action = i->at(3);
     ASSERT_EQ("windows.create", action->api_name());
-    ASSERT_EQ("[{\"url\":\"\\u003Carg_url\\u003E\"}]",
+    ASSERT_EQ("[{\"url\":\"\\u003Carg_url>\"}]",
               ActivityLogPolicy::Util::Serialize(action->args()));
     ASSERT_EQ("http://www.google.co.uk/", action->arg_url().spec());
   }