Revert my recent changes regarding title directionality.

This reverts:
  r82400: Plumb direction of document title through IPC layer.
  r82582: Add and use a base::i18n::StringWithDirection for carrying titles.
  r82778: Change NavigationEntry's title fields to carry the text direction.

I'm going to take an alternative approach; after getting this far, I can
see that this approach was too complicated.

BUG=27094

Review URL: http://codereview.chromium.org/6901003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82908 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 15 insertions, 23 deletions

diff --git a/content/browser/tab_contents/navigation_entry_unittest.cc b/content/browser/tab_contents/navigation_entry_unittest.cc
index 6fbc98f..ee7d083 100644
--- a/content/browser/tab_contents/navigation_entry_unittest.cc
+++ b/content/browser/tab_contents/navigation_entry_unittest.cc
@@ -18,13 +18,11 @@ class NavigationEntryTest : public testing::Test {
     entry1_.reset(new NavigationEntry);
 
     instance_ = SiteInstance::CreateSiteInstance(NULL);
-    entry2_.reset(new NavigationEntry(
-        instance_, 3,
-        GURL("test:url"),
-        GURL("from"),
-        base::i18n::String16WithDirection(ASCIIToUTF16("title"),
-                                          base::i18n::LEFT_TO_RIGHT),
-        PageTransition::TYPED));
+    entry2_.reset(new NavigationEntry(instance_, 3,
+                                      GURL("test:url"),
+                                      GURL("from"),
+                                      ASCIIToUTF16("title"),
+                                      PageTransition::TYPED));
   }
 
   virtual void TearDown() {
@@ -55,26 +53,23 @@ TEST_F(NavigationEntryTest, NavigationEntryURLs) {
 
   EXPECT_EQ(GURL(), entry1_.get()->url());
   EXPECT_EQ(GURL(), entry1_.get()->virtual_url());
-  EXPECT_TRUE(entry1_.get()->GetTitleForDisplay("").is_empty());
+  EXPECT_TRUE(entry1_.get()->GetTitleForDisplay("").empty());
 
   // Setting URL affects virtual_url and GetTitleForDisplay
   entry1_.get()->set_url(GURL("http://www.google.com"));
   EXPECT_EQ(GURL("http://www.google.com"), entry1_.get()->url());
   EXPECT_EQ(GURL("http://www.google.com"), entry1_.get()->virtual_url());
   EXPECT_EQ(ASCIIToUTF16("www.google.com"),
-            entry1_.get()->GetTitleForDisplay("").string());
+            entry1_.get()->GetTitleForDisplay(""));
 
   // file:/// URLs should only show the filename.
   entry1_.get()->set_url(GURL("file:///foo/bar baz.txt"));
   EXPECT_EQ(ASCIIToUTF16("bar baz.txt"),
-            entry1_.get()->GetTitleForDisplay("").string());
+            entry1_.get()->GetTitleForDisplay(""));
 
   // Title affects GetTitleForDisplay
-  entry1_.get()->set_title(
-      base::i18n::String16WithDirection(ASCIIToUTF16("Google"),
-                                        base::i18n::LEFT_TO_RIGHT));
-  EXPECT_EQ(ASCIIToUTF16("Google"),
-            entry1_.get()->GetTitleForDisplay("").string());
+  entry1_.get()->set_title(ASCIIToUTF16("Google"));
+  EXPECT_EQ(ASCIIToUTF16("Google"), entry1_.get()->GetTitleForDisplay(""));
 
   // Setting virtual_url doesn't affect URL
   entry2_.get()->set_virtual_url(GURL("display:url"));
@@ -83,8 +78,7 @@ TEST_F(NavigationEntryTest, NavigationEntryURLs) {
   EXPECT_EQ(GURL("display:url"), entry2_.get()->virtual_url());
 
   // Having a title set in constructor overrides virtual URL
-  EXPECT_EQ(ASCIIToUTF16("title"),
-            entry2_.get()->GetTitleForDisplay("").string());
+  EXPECT_EQ(ASCIIToUTF16("title"), entry2_.get()->GetTitleForDisplay(""));
 
   // User typed URL is independent of the others
   EXPECT_EQ(GURL(), entry1_.get()->user_typed_url());
@@ -157,12 +151,10 @@ TEST_F(NavigationEntryTest, NavigationEntryAccessors) {
   EXPECT_EQ(GURL("from2"), entry2_.get()->referrer());
 
   // Title
-  EXPECT_EQ(string16(), entry1_.get()->title().string());
-  EXPECT_EQ(ASCIIToUTF16("title"), entry2_.get()->title().string());
-  entry2_.get()->set_title(
-      base::i18n::String16WithDirection(ASCIIToUTF16("title2"),
-                                        base::i18n::LEFT_TO_RIGHT));
-  EXPECT_EQ(ASCIIToUTF16("title2"), entry2_.get()->title().string());
+  EXPECT_EQ(string16(), entry1_.get()->title());
+  EXPECT_EQ(ASCIIToUTF16("title"), entry2_.get()->title());
+  entry2_.get()->set_title(ASCIIToUTF16("title2"));
+  EXPECT_EQ(ASCIIToUTF16("title2"), entry2_.get()->title());
 
   // State
   EXPECT_EQ(std::string(), entry1_.get()->content_state());