request review: fix issue 2821 and 6132

chrome/browser/views/options/general_page_view.cc

 // Copyright (c) 2006-2008 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/views/options/general_page_view.h"
 
 #include "base/gfx/png_decoder.h"
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "base/thread.h"
@@ skipping 282 matching lines @@
   return urls;
 }
 
 int CustomHomePagesTableModel::RowCount() {
   return static_cast<int>(entries_.size());
 }
 
 std::wstring CustomHomePagesTableModel::GetText(int row, int column_id) {
   DCHECK(column_id == 0);
   DCHECK(row >= 0 && row < RowCount());
+  // No need to force URL to have LTR directionality because the custom home
+  // pages control is created using LTR directionality.
   return UTF8ToWide(entries_[row].url.spec());
 }
 
 SkBitmap CustomHomePagesTableModel::GetIcon(int row) {
   DCHECK(row >= 0 && row < RowCount());
   if (!entries_[row].icon.isNull())
     return entries_[row].icon;
   return default_favicon_;
 }
 
@@ skipping 766 matching lines @@
     homepage_use_url_textfield_->SetReadOnly(true);
   }
 }
 
 void GeneralPageView::SetDefaultSearchProvider() {
   const int index = default_search_engine_combobox_->GetSelectedItem();
   default_search_engines_model_->model()->SetDefaultSearchProvider(
       default_search_engines_model_->GetTemplateURLAt(index));
 }