"domain" item added to HistoryEntry to allow showing IDN in chrome://history/

chrome/browser/ui/webui/history_ui_unittest.cc

 // 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.
 
 #include "chrome/browser/ui/webui/history_ui.h"
 
+#include <string>
+
 #include "base/strings/utf_string_conversions.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
 struct TestResult {
   std::string url;
   int64 hour_offset;  // Visit time in hours past the baseline time.
 };
 
@@ skipping 104 matching lines @@
     AddQueryResults(test_data, arraysize(test_data), &results);
     BrowsingHistoryHandler::MergeDuplicateResults(&results);
 
     ASSERT_EQ(2U, results.size());
     EXPECT_TRUE(ResultEquals(results[0], test_data[3]));
     EXPECT_TRUE(ResultEquals(results[1], test_data[1]));
     EXPECT_EQ(3u, results[0].all_timestamps.size());
     EXPECT_EQ(1u, results[1].all_timestamps.size());
   }
 }
+
+TEST(HistoryUITest, SetDisplayableDomain) {

[Patrick Dubroy, 2013/08/20 14:26:37]

I think it would be much more valuable to add a test to history_browsertest.js.

+  const std::string key = "displayableDomain";
+  BrowsingHistoryHandler::HistoryEntry history_entry;
+  DictionaryValue dict_value;
+  history_entry.url = GURL("http://xn--d1abbgf6aiiy.xn--p1ai/");
+  history_entry.SetDisplayableDomain(&dict_value);
+  EXPECT_TRUE(dict_value.HasKey(key));
+  string16 str16;
+  dict_value.GetString(key, &str16);
+  const std::string str = base::UTF16ToUTF8(str16);
+  EXPECT_EQ(str, "\xD0\xBF\xD1\x80\xD0\xB5\xD0\xB7\xD0\xB8\xD0\xB4"
+                 "\xD0\xB5\xD0\xBD\xD1\x82.\xD1\x80\xD1\x84");
+}
+